Skip to content

Conversation

@bpark1327
Copy link
Collaborator

@bpark1327 bpark1327 commented Feb 9, 2024

Pull Request Description

Expose HVAC Detailed Performance Data so that we can accommodate cold climate heat pump performance data from the NEEP database or manufacturer's data.

Checklist

PR Author: Check these when they're done. Not all may apply. strikethrough and check any that do not apply.

PR Reviewer: Verify each has been completed.

  • OS-HPXML git subtree has been pulled
  • 301/ES rulesets and unit tests have been updated
  • 301validator.xml has been updated (reference EPvalidator.xml)
  • Workflow tests have been updated
  • Documentation has been updated
  • Changelog has been updated
  • openstudio tasks.rb update_measures has been run
  • No unexpected regression test changes on CI

@bpark1327 bpark1327 self-assigned this Feb 9, 2024
@bpark1327 bpark1327 requested a review from shorowit March 8, 2024 18:12
Copy link
Collaborator

@shorowit shorowit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is getting close, just a couple more things to do.

@bpark1327 bpark1327 marked this pull request as ready for review March 11, 2024 16:40
@bpark1327 bpark1327 requested a review from shorowit March 11, 2024 16:40
Copy link
Collaborator

@shorowit shorowit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job, looks great. We're going to hold off on merging this in to master for now, but the branch can be used for any analysis projects.

@shorowit shorowit changed the title Expose HVAC Detailed Performance Data Expose HVAC Detailed Performance Data inputs Mar 26, 2024
Base automatically changed from latest-os-hpxml to master May 29, 2024 17:14
@shorowit shorowit marked this pull request as draft August 27, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants