-
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
Auto test using github actions #156
Conversation
Betristor
commented
Aug 9, 2023
- Change solver from gurobi to highs
- Configure highs options to be compatiable with HiGHs.jl
- Compare results with example we provided
- Add github action workflow
checkout submodules.
@RuaridhMacd Automatic test using github actions on one-zone case is done, other cases could be appended after one zone test script. I cached julia project for reusage using julia-actions/cache and tried to compare results with example folders. Now the only problem is how to notify users when summary.txt is created. |
@Betristor This is great! I hadn't realized installing HiGHS.jl also installs the HiGHS binary. That makes it much easier. Let me know if you'd like to discuss how to compare results, but I think a simple table which compares the "Total" columns of costs.csv and hsc_costs.csv would be a good start. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of the SmallNewEngland examples work for me locally. The NorthSea example also works but is much slower with HiGHS than with Gurobi.
I'd recommend that we also run ThreeZones_Liquid, as that tests some of the features missing from the OneZone case.
1. Add threezones, liquid and northsea case test 2. Add keywords ['#skip', 'CompatHelper'] to allow test bypassing to save unnecessary github action
Pair condition judgment. #skip.
@RuaridhMacd I found there is no need to install highs binary on github ubuntu machine to perform the tests. And I added other example case to be tested. |