-
Notifications
You must be signed in to change notification settings - Fork 21
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
Update to Julia1.6.7 and Gurobi 9.5 #72
Conversation
This failed on the write-step as GenX is explicitly references versions <1.7.x
@RuaridhMacd I'm always using julia 1.7.2 to debug code and run examples, everything works fine to me except for writing output where version check is implemented. But I extended them to version 1.8 and there is no bug reporting. |
Julia 1.7.x should also work. I went with Julia.1.6.7 because it's the LTS version, so hopefully will be available to new users for a long time. It sounds like we have the same issues in 1.8. There are if-else statements which are hardcoded based on the Julia version which create a DataFrame column (or take some other necessary action) inside the loop, instead of outside and then editing it. Versions 1.8+ aren't accounted for in the if-else, e.g. in write_subsidy_revenue.jl:
Some actions will also be skipped for >v1.8 for the same reason, so I'm not sure any results can be guaranteed to be accurate. The main GenX repo has removed these hard-coded steps, so we can fix it in the next week or two. |
I turned it into |
This avoids conflicts between versions of Julia, particularly between <v1.7 and >=v1.7
That sounds good. I've made some further changes to merge this with the Logging PR. I've added the DOLPHYNJulEnv folder to the .gitignore, as different formats can be used between Julia versions. This was causing my v1.6.7 runs to fail using the environment made using v1.7+ |
Gitignore could be generated using gitignore command line tool in https://www.toptal.com/developers/gitignore. This tool contain many templates for gitignore. I remembered that I made these changes (extend version check into v1.8) in PR #46 but it's not merged so it's fine to implement it here. |
Gurobi 9.5 has some significant improvements over previous versions, so it would be good to ensure it works with Dolphyn.
To do so, we need to update to the latest version of Gurobi.jl (v0.11.3), which in turn requires Julia v1.6 or later.
I have updated the default environment packages to use the updated version of Gurobi.jl and tested everything using Julia v1.6.7, the current long-term support version of Julia.
It would be nice to move to Julia v1.8+ in the future, but certain elements of the output-writing functions are hard-coded to require Julia v1.7 or less. We could update this code or resync the GenX components with the main GenX repo, where they have already resolved this issue.