Create CircEUlar BMT workflow#506
Draft
macflo8 wants to merge 119 commits into
Draft
Conversation
bca6983 to
58b9636
Compare
Contributor
Author
|
Now rebased on the branch for #486. Commits unique to this branch are the ones following, starting with "Ensure items have URNs in StructureFactory.get()". |
6ef741f to
08ed283
Compare
Contributor
Author
|
As described in #523, the changes of the merged PR need some follow-up adaptations. I have just force-pushed with adapted commit messages. The remaining required changes will follow with the next commits. |
08ed283 to
3b03d02
Compare
Handle items from MARK via @pytest.mark.key, where possible.
- Use via @pytest.mark.key_of_item, instead of importing MARK.
- Extend top-level testing.MARK using transport.testing.MARK, instead of making the latter a ChainMap of the former. - Adjust usage.
- 2 → "R12_only". - 3 → "ISR_no_data". - 4 → "GEA_not_implemented", "SHAPE_not_implemented". - 9 → "R14_no_data". - 10 → "transport_build_data". - Remove unused key 0. - Use "from pytest import mark" for brevity. - Expand type hints.
- Use in message_ix_models.testing hooks.
- Add "no_data", "non_public_data" package-wide marks.
- Use factories through @pytest.mark.no_data("…").
- Picked from #462.
- Picked from #474.
- Extend .buildings.Config - Add .buildings.build.prepare_data_C - Adapt project.circeular.workflow.configure - Test .prepare_data_C & .materials
- Keep power sector LCA mapping pre f7e89f9 - Add new mapping as separate input file - Adjust read_material_intensities to read compatible mapping
- Include material reconciliation in power sector materials build - Add function to update power sector material demand estimate on actual solution
- Update METHOD.C to run STURM dynamically - Update METHOD.C to use circeular scenario code - Fix scenario code propagation to STURM
Add module for GLOMIS soft-linking
- Retrieve platform name from context - Use CircEUlar specific STURM input file names
Fix wrong use of `cap_comm` argument
- Skip independent reporting steps - Add and rename power and infrastructure build steps
- Add new input file - Update circeular workflow config
Add duplicates with CircEUlar scenario labels as filename Required to get correct parametrization for CircEUlar scenarios
018cb12 to
06932b5
Compare
- Fix order of args for buildings materials subtraction - Drop negative values after power sector material reconciliation - Fix GLOMIS build
Copy from .project.ngfs from branch project/ngfs_p6_bmt
a303ec2 to
c591926
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TBA
How to review
TBA
PR checklist