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

Consolidate curtailment calculations #260

Open
jaredthomas68 opened this issue Dec 15, 2023 · 0 comments
Open

Consolidate curtailment calculations #260

jaredthomas68 opened this issue Dec 15, 2023 · 0 comments

Comments

@jaredthomas68
Copy link
Collaborator

Consolidate curtailment calculations

Currently, curtailment is calculated in many locations and this creates a lot of unnecessary code that is untested and difficult to read. I believe that most instances, if not all, should be using the same function.

Proposed solution

We could reduce excess code by creating a single curtailment function that all cases of curtailment calculations call on.

Alternatives considered

Keep it as is.

Additional context

When addressing greenheart we may need to curtail products besides electricity (hydrogen, ammonia, etc). So the curtailment would ideally be written generally to work in all use cases.

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

No branches or pull requests

1 participant