Repository navigation
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## graph_multi_region #1575 +/- ##
===================================================
Coverage 90.90% 90.90%
===================================================
Files 61 61
Lines 9116 9116
Branches 9116 9116
===================================================
Hits 8287 8287
Misses 507 507
Partials 322 322 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
tsmbland
marked this pull request as ready for review
November 3, 2026 08:52
This branch has not been deployed
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.
Description
I thought it would be useful to have a section in the documentation on commodity graphs, as this is an important part of how MUSE works, and a fundamental improvement over MUSE1 where users had to manually define hierarchies. Rather than writing this manually, I tried to get copilot to do it mostly for me. This is based off #1569, where region-level graphs have been combined.
My process was as follows:
I want a section in the user-facing docs describing commodity graphs in the context of the model. It should mention the fact that graphs are built and why:
The docs should focus on user facing implications, rather than algorithmic details. The goal is to help users to build models and diagnose problems
A few things to mention (allows users to understand visualised graphs and diagnose problems)
There should be at least one diagrammatic example, maybe more. E.g. we should give an example of a set of commodities and processes, and how the nodes in the graph represent commodities and the edges are processes. For simplicity, limit this to one region (don't include regions on the graph). Colour code by SVD/SED/OTH. Include a circularity. Then from the graph, write down/show what the investment order would be, walking it through. It might be easier to give dot code and I go away and create a png/svg - not sure. Use the example models for inspiration.
This documentation should be easy to read for a MUSE newcomer. The aim is to educate users to help them use MUSE effectively, not to confuse them. Aim for conciseness and do not include unnecessary details that aren't important for MUSE users to know.
Come up with a detailed plan
Notes:
circularityexample is a bit too complex for this.save-graphscommand more visually appealing. The first graph in the doc is good inspiration. If we did this then maybe we could combine these sections. Beyond the scope of this PR though.Fixes # (issue)
Type of change
Key checklist
$ cargo test$ cargo docpresent in the previous release
Further checks