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 data for simple investment method #1016

Closed
gnawin opened this issue Feb 6, 2025 · 4 comments
Closed

Fix data for simple investment method #1016

gnawin opened this issue Feb 6, 2025 · 4 comments
Assignees

Comments

@gnawin
Copy link
Member

gnawin commented Feb 6, 2025

Description

Now that we get rid of the simple investment method, the data given for it needs to be reviewed and modified to align with what is expected now (in the same format with compact).

@gnawin gnawin self-assigned this Feb 6, 2025
@gnawin
Copy link
Member Author

gnawin commented Feb 6, 2025

Currently the simple method is used for battery, solar, and ocgt.

  • battery has a lifetime of 15 years, investable is true for both 2030 and 2050. In asset_both, there are rows for (2030,2030) and (2050,2050) with decommissionable false. There is also initial units from 2020, it appears in (2030,2020) with decommissionable false. In 2050, it is beyond its lifetime, so there is no row of (2050,2020).
  • solar has a lifetime of 15 years, investable is true for both 2030 and 2050. In asset_both, there are rows for (2030,2030) and (2050,2050) with decommissionable false.
  • ocgt has a lifetime of 15 years, investable is true for both 2030 and 2050. In asset_both, there are rows for (2030,2030) and (2050,2050) with decommissionable false.

In assets-profiles, there are profiles for solar 2030 and 2050. No profiles for battery and ocgt.

So it seems there is nothing wrong in the data, could you see what's wrong there @abelsiqueira ?

@abelsiqueira
Copy link
Member

Hi @gnawin, here is the link to the model files:
https://gist.github.com/abelsiqueira/67704e0e32db45110093c37f8f73d3c4
There is a Zip download button in the top right corner, if that helps.

@datejada
Copy link
Member

I think this issue is closed with #1004, @gnawin could you please double-check, and if so, then please close this issue. Thanks!

@gnawin
Copy link
Member Author

gnawin commented Feb 18, 2025

Yes, this is no longer relevant.

@gnawin gnawin closed this as completed Feb 18, 2025
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

3 participants