8000 Addresses inconsistencies by shorowit · Pull Request #124 · hpxmlwg/hpxml · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Addresses inconsistencies #124

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

Merged
merged 4 commits into from
Jul 26, 2019
Merged

Addresses inconsistencies #124

merged 4 commits into from
Jul 26, 2019

Conversation

shorowit
Copy link
Contributor
@shorowit shorowit commented Jun 20, 2018

Addresses several inconsistencies as documented in #111. Also fixes #29.

  • Renames HeatPump/AnnualHeatEfficiency to HeatPump/AnnualHeatingEfficiency (now consistent with HeatingSystem/AnnualHeatingEfficiency)
  • Renames HeatPump/AnnualCoolEfficiency to HeatPump/AnnualCoolingEfficiency (now consistent with CoolingSystem/AnnualCoolingEfficiency)
  • Replaces Measure/InstalledComponent with Measure/InstalledComponents/InstalledComponent (now consistent with Measure/ReplacedComponents/ReplacedComponent)
  • Renames FoundationWall/BelowGradeDepth to FoundationWall/DepthBelowGrade (now consistent with Slab/DepthBelowGrade)
  • Replaces WeatherStation/SystemIdentifiersInfo with WeatherStation/SystemIdentifier (now consistent with all other elements)
  • Renames "central air conditioning" to "central air conditioner" for CoolingSystemType (now consistent with "room air conditioner")
  • Renames HeatPump/BackupAFUE to BackupAnnualHeatingEfficiency, accepts 0-1 instead of 1-100 (now consistent with AnnualEfficiency elements)

@shorowit shorowit requested a review from nmerket July 2, 2019 15:09
@shorowit
Copy link
Contributor Author
shorowit commented Jul 2, 2019

Check BackupHeatEfficiency vs BackupHeatingEfficiency

@shorowit
Copy link
Contributor Author
shorowit commented Jul 2, 2019

Double-checked, it is "BackupAnnualHeatingEfficiency". I think this is good to go.

@shorowit shorowit added the merge label Jul 2, 2019
@nmerket nmerket merged commit a635ac9 into master Jul 26, 2019
@nmerket nmerket deleted the inconsistencies branch July 26, 2019 16:42
@nmerket nmerket added this to the v3.0 milestone Oct 30, 2019
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.

HeatPump/BackupAFUE using 1-100 AFUE instead of 0-1
2 participants
0