-
Notifications
You must be signed in to change notification settings - Fork 23
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
Make 3 object aggregates subclasses of (supply) system #1833 #1947
base: dev
Are you sure you want to change the base?
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.
Looks good, thanks. Just some minor things.
I am wondering, why there are so many changes that are no actual changes. This makes the review confusing. I thought we had fixed that. Did you pull from the latest dev version?
I branched this while we still had the bug with the doubles lables. I think that's where this problem came from. Maybe I should have merged dev into it again and that would have avoided the problem. I applied your changes. I feel like I always have trouble knowing how brief to be in changelog |
Co-authored-by: stap-m <[email protected]>
Co-authored-by: stap-m <[email protected]>
Co-authored-by: stap-m <[email protected]>
Co-authored-by: stap-m <[email protected]>
Summary of the discussion
See #1833.
As agreed in the discussion
supply grid
,transport network
andcritical infrastructure
should besupply systems
instead ofobject aggregates
. I changed the definitions accordingly.Please let me know if the equivalence for
critical infrastructure
should change (as also asked in #1833)Type of change (CHANGELOG.md)
Update
Workflow checklist
Automation
Closes #1833
PR-Assignee
term tracker annotation
Reviewer