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

Enable CustomFinancialModel to get costs from PySAM #264

Open
jaredthomas68 opened this issue Jan 16, 2024 · 1 comment
Open

Enable CustomFinancialModel to get costs from PySAM #264

jaredthomas68 opened this issue Jan 16, 2024 · 1 comment
Assignees

Comments

@jaredthomas68
Copy link
Collaborator

The financial model must be consistent across all technologies, but the wave energy requires the CustomFinancialModel class. HOPP uses the SingleOwner financial model to get PV (and other tech) costs. Thus, when using CustomFinancialModel the PV costs are not available.

We need to update HOPP such that PySAM costs can be accessed when using the CustomFinancialModel class. This could be possibly done by inheriting from the SingleOwner class or by creating methods in CustomFinancialModel to access PySAM costs.

@kbrunik
Copy link
Collaborator

kbrunik commented Jan 30, 2024

Addressed in jaredthomas68#2

  • Creates new method set_overnight_capital_cost() in PowerSource to add technology costs.
  • Modified set_overnight_capital_cost() for Battery class, includes energy_capital_cost and power_capital_cost

Ideally in the future we can pull ATB costs directly from an API or other method. There is an outdated ATB json in HOPP (2020), could potential update to most recent ATB but not sure if that's the best path.

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

2 participants