-
Notifications
You must be signed in to change notification settings - Fork 49
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
Update tag_residential_loadshifting_assets.yaml #192
Conversation
Add Electric Vehicle_55 which represents expectations for EV energy demand using the Fit for 55 scenario
Change of description of refrigeration, and add Circulation Pump asset
@@ -6,6 +6,8 @@ | |||
- Residential Load Shifting Asset: | |||
- Electric Vehicle: | |||
description: electric vehicles | |||
- Electric Vehicle_55: |
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 would not add a new variable for describing soomething which is the same than the previous variable but for a different ""scenario""; my suggestion would be to use the column "Scenario" i the IAMC format and include there the different scenario names; so I suggest to remove this tag from the nomenclature
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.
@danielhuppmann do you agree with what I wrote just above?
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.
Yes, I agree - this is not the intended usage of a variable. Please remove or close this PR and start a new one that has only the changes to the other descriptions.
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.
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.
suggestion to remove electric vehicle_55
Co-authored-by: sandrinecharousset <62063474+sandrinecharousset@users.noreply.github.com>
Co-authored-by: sandrinecharousset <62063474+sandrinecharousset@users.noreply.github.com>
Add Electric Vehicle_55 which represents expectations for EV energy demand using the Fit for 55 scenario