Skip to content
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

BuildResidentialHPXML: add loop/coil watts for shared boiler #1616

Closed
joseph-robertson opened this issue Feb 19, 2024 · 4 comments · Fixed by #1621
Closed

BuildResidentialHPXML: add loop/coil watts for shared boiler #1616

joseph-robertson opened this issue Feb 19, 2024 · 4 comments · Fixed by #1621
Assignees
Labels
bug Something isn't working

Comments

@joseph-robertson
Copy link
Collaborator

Without the ability to specify shared_loop_watts and/or fan_coil_watts, we can no longer produce valid HPXML files generated by ResStock when sampling a shared boiler.

@joseph-robertson joseph-robertson added the bug Something isn't working label Feb 19, 2024
@shorowit
Copy link
Contributor

Perhaps ResStock should not use the shared boiler options? See NREL/resstock#1139.

But yes, I agree that this is a problem. There is no way to fully describe these systems anymore. Possible solutions:

  • Add the two inputs as you suggested
  • Revert the previous changes where shared boilers no longer have their auxiliary energy defaulted
  • Remove the ability to describe shared boilers in the BuildResHPXML measure altogether

@lixiliu
Copy link
Collaborator

lixiliu commented Feb 19, 2024

Can we go with option 1? Not sure what option 2 is. I don't think we should remove shared boilers from BuildResHPXML measure. FWIW, 10% of the housing stock has a shared heating/cooling system and we still don't capture all the diversity we need for shared systems.

@afontani
Copy link
Collaborator

I would like to continue to support shared heating and cooling systems and find a way to give them the attention they need to be more representative of systems in real buildings. There are ResStock users (users of the data) who want better shared system models. The Building Performance Standards also cover these systems which the data we have is not great.

To Lixi's point, for Multi-Family units, shared HVAC systems are fairly common.
Screenshot 2024-02-19 at 4 48 25 PM

I take it that the shared_loop_watts and the fan_coil_watts inputs were for the energy used to manage the fans and pumps for the shared system?

I think my preference would be to go with adding the variables to capture the energy. I think. I would rather have the ability to capture the energy.

@shorowit shorowit moved this from Triage to High Priority in OpenStudio-HPXML Feb 20, 2024
@afontani
Copy link
Collaborator

After some discussion, we are leaning towards the 2nd bullet: Revert the previous changes where shared boilers no longer have their auxiliary energy defaulted.

@shorowit shorowit self-assigned this Feb 21, 2024
@shorowit shorowit moved this from High Priority to In progress in OpenStudio-HPXML Feb 21, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in OpenStudio-HPXML Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants