-
Notifications
You must be signed in to change notification settings - Fork 34
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
Change technology inputs and replace solar + wind technologies in tools.costs
#206
Conversation
@measrainsey here I'd suggest "Update branch" → "Update with rebase". Should be no conflicts. |
303fbf8
to
48454fd
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #206 +/- ##
=====================================
Coverage 65.5% 65.5%
=====================================
Files 204 204
Lines 15822 15822
=====================================
Hits 10378 10378
Misses 5444 5444
|
Sorry, I forgot to check the tests 🤦🏻♀️ Turns out they were failing because message-ix-models/message_ix_models/data/technology.yaml Lines 594 to 606 in 48454fd
Because I added these new technologies, I also had to update some of the tests in All checks passing now, but let me know if my changes might be problematic. |
Just a note that I added some more technologies: 2005, 2010, 2015, and 2025 bins for the |
7fcefdc
to
1f166a7
Compare
7373735
to
6c37f66
Compare
b3af303
to
f3d379a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small question, otherwise, this looks good to me (after rebasing onto current main
:)
doc/whatsnew.rst
Outdated
@@ -16,6 +18,7 @@ v2024.8.6 | |||
- Add :doc:`/project/edits` project code and documentation (:pull:`204`). | |||
- Reduce log verbosity of :func:`.apply_spec` (:pull:`202`). | |||
- Fix and update :doc:`/api/tools-costs` (:pull:`186`, :pull:`187`, :pull:`190`, :pull:`195`). | |||
- Made fixes and updates to :doc:`/api/tools-costs` (:pull:`186`, :pull:`187`, :pull:`190`, :pull:`195`, :pull:`206`). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we get this new line instead of expanding the one above it (using "Fix und update ...")?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the catch :) Think I made an error within doc/whatsnew along the way by adding this PR #206 to the v2024.8.6 release instead of under "Next release". I deleted this entry under v2024.8.6, but this PR is documented under "Next release".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We also have this line:
message-ix-models/.gitattributes
Lines 1 to 2 in 32cb2fd
# Reduce the number of merge conflicts | |
doc/whatsnew.rst merge=union |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks all - after a few mistakes I think I sorted the doc/whatsnew
issues. Took a few commits to get there but hope that's fine.
For the subsequent PRs #221, #227, and #222, I rebased and dropped all the other commits related to doc/whatsnew because they were creating some clutter. Then just did one final edit to doc/whatsnew for each PR.
a727124
to
621611d
Compare
621611d
to
faa43b7
Compare
faa43b7
to
00d4120
Compare
@measrainsey @glatterf42 from my POV this is (a) up to date with rebase and (b) approved. Can we now merge? |
Yep - if all good with everyone, I'm happy to merge. Thanks! |
…y technologies Main changes are: - update `stor_ppl` to use the same scenario categories as `solar_pv_ppl` - update `csp_sm1_ppl` and `csp_sm3_ppl` to borrow from `solar_th_ppl` as starting point - for many `_i` and `hpl` technologies, change mapping of WEO technologies - change hydrogen to be mapped to `fuel_cell` in WEO
Copied over the values (base year mapping, first year, cost reduction rates, and scenario mappings) from solar_pv_ppl for new solar technologies: - solar_res1 - solar_res2 - solar_res3 - solar_res4 - solar_res5 - solar_res6 - solar_res7 - solar_res8 - solar_res_hist
Copied over input data for wind_ppl and added new technologies: - wind_res1 - wind_res2 - wind_res3 - wind_res4 - wind_res_hist
Copied over input data for wind_ppf and replaced with the following new technologies: - wind_ref1 - wind_ref2 - wind_ref3 - wind_ref4 - wind_ref5 - wind_ref_hist
- Replace `solar_pv_ppl` with `solar_res1` - Replace `wind_ppl` with `wind_res1`
…ata/technology.yaml
00d4120
to
1788b18
Compare
Rebased to resolve conflicts in |
Make changes to the energy module input technologies
All changes made in this PR are contained to the input CSVs, so no changes to the code. The main changes are:
Change inputs for some storage, CSP, hydrogen, and industry technologies
cc @volker-krey
The following changes were made to the input CSVs:
stor_ppl
to use the same scenario categories assolar_pv_ppl
csp_sm1_ppl
andcsp_sm3_ppl
to borrow fromsolar_th_ppl
as starting point_i
andhpl
technologies, change the mapping of WEO technologiesfuel_cell
in WEOReplace existing wind and solar technologies with new technologies
cc @OFR-IIASA
As part of the SSP update process, solar and wind technologies are being replaced with new (more) technologies. I have copied over the inputs/assumptions from the previous technologies and used them for the new technologies.
solar_pv_ppl
has been replaced with:solar_res_hist_2005
solar_res_hist_2010
solar_res_hist_2015
solar_res_hist_2020
solar_res1
solar_res2
solar_res3
solar_res4
solar_res5
solar_res6
solar_res7
solar_res8
wind_ppl
has been replaced with:wind_res_hist_2005
wind_res_hist_2010
wind_res_hist_2015
wind_res_hist_2020
wind_res1
wind_res2
wind_res3
wind_res4
wind_ppf
has been replaced with:wind_ref_hist_2005
wind_ref_hist_2010
wind_ref_hist_2015
wind_ref_hist_2020
wind_ref1
wind_ref2
wind_ref3
wind_ref4
wind_ref5
How to review
For @khaeru and/or @glatterf42 : Read the diff and note that the CI checks all pass.
PR checklist
Add or expand tests;Modified tests; coverage checks both ✅