-
Notifications
You must be signed in to change notification settings - Fork 155
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
Updated AEI initialization to Mehta (2024) #672
base: develop
Are you sure you want to change the base?
Updated AEI initialization to Mehta (2024) #672
Conversation
validation = "rev4.104_h12_validation.tgz", | ||
cfg$input <- c(regional = "rev4.106_h12_magpie.tgz", | ||
cellular = "rev4.106_h12_fd712c0b_cellularmagpie_c200_MRI-ESM2-0-ssp370_lpjml-8e6c5eb1.tgz", | ||
validation = "rev4.106_h12_validation.tgz", |
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.
I'm always a bit unsure, these datasets should also be put into scenario_config and the fsec scenario config, right?
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.
Yes, absolutely! Thanks for pointing this out!
I'm currently waiting if/how we'll solve the warnings in the preprocessing. Once this is solved, I can set up a new preprocessing for both default as well as the FSEC scenario config.
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 are also still waiting for Patrick's PR. It'll have to be merged before mine because it also contains changes in the preprocessing that affect the GAMS code.
things were still changed after the review, please request another review once the PR is ready :)
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.
input data still needs to be set, otherwise looks good :)
Please also update |
Yes. This is what comes in Patrick's PR. |
Yes, this is what comes in Patrick's PR. |
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.
Generally looks good to me. Please see my minor comments and also update the input data.
config/default.cfg
Outdated
cfg$input <- c(regional = "rev4.105_h12_magpie.tgz", | ||
cellular = "rev4.105_h12_fd712c0b_cellularmagpie_c200_MRI-ESM2-0-ssp370_lpjml-8e6c5eb1.tgz", | ||
validation = "rev4.105_h12_validation.tgz", | ||
cfg$input <- c(regional = "WARNINGS66_rev4.107Mehta_h12_magpie.tgz", |
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.
The rev4.106 preprocessing is ready now :)
🐦 Description of this PR 🐦
🔧 Checklist for PR creator 🔧
Label pull request from the label list.
Self-review own code
magpie4
R library has been updated accordingly and backwards compatible where necessary.scenario_config.csv
has been updated accordingly (important ifdefault.cfg
has been updated)Document changes
CHANGELOG.md
goxygen::goxygen()
and verify the modified code is properly documentedPerform test runs
Rscript start.R --> "compilation check"
Rscript start.R --> "test runs"
Rscript start.R --> "test runs"
📉 Performance changes 📈
🚨 Checklist for reviewer 🚨
CHANGELOG
is updated correctly