-
Notifications
You must be signed in to change notification settings - Fork 0
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
format scenario data for p4b input #58
Conversation
I agree that we should avoid polluting Perhaps the 3 of us can have a discussion on the best way to approach this? |
yea, we had an initial discussion last week but now that the code is functional, I would appreciate a follow up for a decision. I want to deliver the p4b scenarios this sprint and while I have them prepared locally, I would feel better about using the final script for that. |
Docker build status
|
I wasn't expecting it to require so much code. Might make sense to put it in a separate script source it, same as the separate scenario scripts. |
lookth good |
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.
a bit of a rubber stamp on the changes made since I last reviewed this, but I trust this works
depends on: RMI-PACTA/pacta.scenario.data.preparation#59
format_scenarios_for_p4b.R
which formats the prepared scenarios for use in P4Bscenarios_to_include
from config, which is determined by theconfig_name
(i.e. the timestamp of the data preparation)p4b_scenarios_YYYY.csv
andp4b_ei_scenarios_YYYY.csv
in the output_path does not interfere with workflow.data.preparation, because scenario files in that workflow are specifically targeted following thesource_YYYY.csv
naming convention, see https://github.com/RMI-PACTA/workflow.data.preparation/blob/9beb13c4bdce7937bcfe327e8e12d75599090dcc/run_pacta_data_preparation.R#L142 specifying scenario filenames in the config as shown here: https://github.com/RMI-PACTA/workflow.data.preparation/blob/9beb13c4bdce7937bcfe327e8e12d75599090dcc/config.yml#L176final_year
as a cutoff for the scenarios.final_year
is determined as the maximum year across featured scenario sources. For the preparation of 2023 scenarios, this is 2050. Values post 2050 seem a bit uncertain in GECO, so this was determined as the best way forward for now.NOTE: