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

Energy Score Changes #65

Merged
merged 2 commits into from
Jun 22, 2016
Merged

Energy Score Changes #65

merged 2 commits into from
Jun 22, 2016

Conversation

nmerket
Copy link
Contributor

@nmerket nmerket commented Nov 30, 2015

Adding OtherScoreType and ScoreDate. Does this work for everyone?

baseelements_energyscoretype

@brandongallagher @kmwoley @GamalielL @juliecaracino @RebHudson @gmdickey

Fixes #39
Fixes #40

@nmerket nmerket added this to the v2.2 milestone Nov 30, 2015
@brandongallagher
Copy link

That works for me.

@nmerket
Copy link
Contributor Author

nmerket commented Dec 3, 2015

Check for alignment with RESO Data Dictionary.

@nmerket
Copy link
Contributor Author

nmerket commented Dec 17, 2015

In the RESO data dictionary v1.4, they have not defined any enumerations for GreenBuildingVerificationType, their congruent field. They are in the process of determining the list for the v1.5 release. That is still being drafted. I'm working with the right people to make sure we're part of that conversation.

@nmerket
Copy link
Contributor Author

nmerket commented Feb 4, 2016

@nmerket talk to Rob Larsen at RESO.

@nmerket
Copy link
Contributor Author

nmerket commented Jun 13, 2016

This matches up with the scores that RESO has (Home Energy Score and HERS Index). They have other GreenBuildingVerificationTypes but that information is stored elsewhere in HPXML.

@nmerket
Copy link
Contributor Author

nmerket commented Jun 13, 2016

Basically the only change here is that we now have an OtherScoreType and a ScoreDate elements.

@nmerket nmerket merged commit 3f7827f into master Jun 22, 2016
@nmerket nmerket deleted the energy_score_changes branch June 22, 2016 20:14
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.

2 participants