-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing Constraint #4
Comments
It's not in there, if you want it you'll have to add it yourself. But from my understanding of it, it won't be feasible in most cases. |
I have made several attempts to create the constraint, but I'm still new to programming and only found out about Julia when I discovered the MIT project these programs were built for. It seems the way objects are created is a lot different than Python or something. This is what I have so far: Another attempt: I'm not sure if I'm even close to getting it right. I think the constraint needs to use :Batting_Order_Confirmed_ but it gave me an error saying it was undefined at one point. On the second example, it says "i" is undefined. I've been adding this into the baseball_formulations.jl file with the other constraints. Can someone please help? |
jnederlo - Would making these changes accomplish what I'm trying to do? Assume we use the 1-6 batters. I'm new to Julia, but it seems like this would work if the way I'm reading the program is right. Definitely possible I'm not though. In baseball_formulations.jl - In data_cleaning.jl -
I made these changes and the program ran without any errors. What do you think? |
I'm not sure, I'm not very familiar with circshift, it's possible what you've done worked, I would manually check with your outputs. But as I mention above, the much simpler way to me seems to just filter your inputs altogether, i.e. filter the data before you send it into julia. |
line 30 #only keep 4th order and earlier batters, cuz they make more points
On line 30 of the baseball_formulations program, you mention the batting order constraint, but I did not see the actual constraint in any of the programs. If the constraint is there, please tell me where. I'd like to adjust it. Otherwise, can you please provide the constraint that is missing?
The text was updated successfully, but these errors were encountered: