-
Notifications
You must be signed in to change notification settings - Fork 256
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
feat: check objective in test routines #1468
Conversation
Validator ReportI am the Validator. Download all artifacts here. ❗ Run failed!Download 'logs' artifact to see more details.
Model Metrics Comparing |
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.
Good utility!
Would this have catched the linopy bug? I guess no since the test configs are to small? |
Co-authored-by: Fabian Neumann <[email protected]>
I think so yes! the exclusion of balance contraints which happened was altering the optimization quite significantly. |
Option to counter-check the objective value against expected value. With this check, we make sure that no strong deviations from the expected objective are introduced. In case the objective varies stronger than the allowed tolerance, but for a good reason (new tech/tech costs), the expected value has to be updated manually.
Changes proposed in this Pull Request
Checklist
envs/environment.yaml
.config/config.default.yaml
.doc/configtables/*.csv
.doc/data_sources.rst
.doc/release_notes.rst
is added.