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
Improve net-metering big-m constraint #383
Conversation
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.
@zolanaj could you add an update to the Changelog please?
post-discussion changes:
|
Mac tests seem to be stalling out but things are working fine in the Windows test suite! |
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.
Thanks so much for implementing this! Strange that the Mac tests aren't passing quickly after your test improvements, but it seems like that issue is not directly related to this PR so I am approving
Fixed
Improved the practical upper bound on interconnect limit for net metering constraints. Prevents erroneous results in which net metering limit is exceeded and net metering benefit is still gained.
Added
New tests for net metering added.
one comment: there's a max benefit of 10 times the annual energy charges which is reached if the wholesale rate provided is larger than the energy price. One question is: should we throw a warning if the results hit this max benefit?