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
Remove redundant items #329
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.
This is great, thanks Jie! A couple things:
- we need to base the PR against a new
develop-v3
branch which I will create today. - Can you add a proposal into the proposal folder so that we know what this change is?
- After 2, can we also include Jan and Michael Braunch from CTS on these updates?
Will do! |
Hey Nick, please check the added proposals for the changes. |
Great, thanks! I added Yan as a reviewer. Can you send link to CTS? I'll slack you the email. |
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. Let's wait until after meeting with CTS before merging.
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.
Make decision and implement the latest proposal: delete Install or upgrade master venting
|
||
## Justification | ||
|
||
`Install or upgrade master venting` measure repeats in `ChilledWaterHotWaterAndSteamDistributionSystems` and `OtherHVAC`, and we should keep only one of them. This proposal suggests that we remove it under `ChilledWaterHotWaterAndSteamDistributionSystems` and keep it under `OtherHVAC`, as master venting approach could be implemented not only in chilled or hot water system. |
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.
@nllong could we justify whether we should keep it under ChilledWaterHotWaterAndSteamDistributionSystems
or OtherHVAC
? Any reference to look up?
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.
Reference issues #271, #272, #275