Skip to content
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

Fix lack of deficit objective term #18

Closed
wants to merge 1 commit into from

Conversation

andrewrosemberg
Copy link
Member

@andrewrosemberg andrewrosemberg commented Mar 15, 2024

Deficit slack is not being penalized in the objective function.

At some point, our code to add a deficit (load shedding) slack stopped working. I remember it working when I was running experiments, but now it no longer works.

This is a testament to the negative effects of bad testing in the package (my bad).

@andrewrosemberg
Copy link
Member Author

nevermind, this has something to do to how the model is built. I was testing at the wrong time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant