-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix: full run with system v2 components #147
Conversation
af06892
to
91fb95a
Compare
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.
LGTM!
Calculate emissions for fuel consumers and emitters | ||
:param variables_map: | ||
:param consumer_results: | ||
:return: |
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.
I think we agreed to use googles doc-string format
"typ": "STREAM_DAY", | ||
"unit": "NOK/d", | ||
"values": [ | ||
0.0, |
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.
0 co2?
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.
0 quota for co2, 85.8 tons co2 (rate below)
91fb95a
to
c766174
Compare
Why is this pull request needed?
ecalc model with system v2 did not run
What does this pull request change?
Issues related to this change: