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

Feature request: unify om_capacity parameter #401

Open
jaredthomas68 opened this issue Nov 8, 2024 · 0 comments
Open

Feature request: unify om_capacity parameter #401

jaredthomas68 opened this issue Nov 8, 2024 · 0 comments

Comments

@jaredthomas68
Copy link
Collaborator

unify om_capacity parameter across all tech including battery

As a holdover from the initial implementation of the custom financial model, and I believe from pySAM, the battery financial has a different name for the om_capacity, om_batt_capacity_cost.

Proposed solution

Ideally we would have a unified name for this, just om_capacity or om_capacity_cost regardless of technology.

Alternatives considered

Currently both om_capacity and om_batt_capacity_cost are set for battery.

Additional context

Some discussion on this from PR #355. @dguittet originally implemented the custom financial model and will likely have some insight on a good solution for HOPP.

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