-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add offshore nodes to YAML #3
Conversation
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.
See a first round of comments inline.
Co-authored-by: Daniel Huppmann <[email protected]>
Co-authored-by: Daniel Huppmann <[email protected]>
Co-authored-by: Daniel Huppmann <[email protected]>
Co-authored-by: Daniel Huppmann <[email protected]>
Co-authored-by: Daniel Huppmann <[email protected]>
Co-authored-by: Daniel Huppmann <[email protected]>
In the results data, it is called "The Netherlands", should it be like this here as well then? Or does that not matter? |
For consistency across the project, the region in the database has to be in line with the spelling in the There are two options to get that result:
Given that you will also want to report directional results, it may be easier to use the second approach. |
Okay! Thank you! I will change it in the output file. |
Hi Daniel, I implemented the changes now and hope that everything is correct. Do I open a new PR then or can I renew this one? |
No need to open a new PR, just committing to a branch with an open PR and pushing to Github is sufficient. |
Okay. That should be done if I am not mistaken. |
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!
Two questions:
- Do you want to have region-processing from country-level to European aggregates? Or are you going to include all relevant aggregates in your scenario-submission files? If you want to use region-aggregation processing as part of the submission to the Scenario Explorer database (and have smaller files for the upload), you should add a mapping file.
- Are there other disaggregations of the North Sea areas? Or will all models in iDesignRES that have such level of detail use the same regional disaggregation. Your specification of territories of "North Sea" in the file
european-regions.yaml
implies that this is the one and only disaggregation to be used. If that is indeed the case, it would make sense to define the different subdivisions of the North Sea there, and not make them specific to EMPIRE.
Also, please remove the file connections.yaml
for now. As a first test upload of a scenario, having the countries and subdivisions will be sufficient. Once you have successfully uploaded such a file, we can add scenario results on interconnections.
Also, we will work on additional validation features so that items like "NO1>NO2" are not allowed (I assume that these should be renamed to proper North Sea areas).
Hi Daniel, Thanks for your reply. I have to double check this and discuss it with Mostafa. My task is to upload to EMPIRE-data from the run I did but I am not sure about the project-wide details regarding these questions, so I don't want to give a wrong answer here. I'll get back to you once I know more. Another question: Best, Jannis |
Hi Daniel, I will check for the aggregations in the data now. |
The discussion we are having here is the model registration. Once we have agreed on the region-naming and merged this PR, you can submit scenarios.
The region-names are "EMPIRE v2|Norway|Ostland", etc. - not NO1. Please go through that list carefully and double-check that you are using the correct region names.
Let's keep this as "EMPIRE v2|North Sea|..." for now. |
Hi Daniel, Sorry for having to annoy you with this so much. I was asked to rename the model run back to EMPIRE1.0 and then only change the scenario for the name. Is that possible? Best, Jannis |
Yes, this is possible, but I advise to use different version numbers when the regional resolution of a model changes. For example, this would make it very confusing when using the region-processing of the Scenario Explorer infrastructure. |
Mostafa told me to ask if we could go for model name: V1.0.0/v51 . Is that fine? |
Why? |
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.
Sorry for being a pain but the region-definitions have to be correct...
Regarding the model name/version: |
Ok, then please rename all regions in this PR from "EMPIRE v2" to "EMPIRE V1.0.0/v51" if that's what you want. |
Co-authored-by: Daniel Huppmann <[email protected]>
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.
Sorry...
definitions/region/connections.yaml
Outdated
@@ -0,0 +1,193 @@ | |||
- Connections: | |||
# allow directional connections explicitly | |||
- Austria>Czechia |
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 (directional) region names following the hierarchy have to be indented, like in the example shown in the other thread.
- Austria>Czechia | |
- Austria>Czechia |
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.
Thank you, tests are passing!
Hi Daniel,
As discussed, I added the offshore nodes to the YAML. I hope the way I did it is correct.
Best,
Jannis