Latest OS-HPXML

[joseph-robertson]

Pull Request Description

Pulls in:

• BuildResidentialHPXML: update apply_defaults and apply_validation OpenStudio-HPXML#1467
• Exclude None from default_na_values list when reading csv files OpenStudio-HPXML#1466
• Better workflow test warning checks OpenStudio-HPXML#1469

Checked means that something in resstock was changed or updated as a result of pulling in.

Questions:

• Should we re-think when/where validation on the HPXML file is performed? Currently we skip validation from HPXMLtoOpenStudio. Instead, we validate both before and after applying defaults from BuildExistingModel and ApplyUpgrade. (We apply defaults from ApplyUpgrade so that home.xml looks similar to the home.xml produced when there are no upgrades.) Validation therefore happens 2 times for a home with no upgrade, and 4 times for a home with an upgrade. Should we switch back to having HPXMLtoOpenStudio do the validation? Then we would only be validating 2 times (when no upgrade) or 3 times (when upgrade).

Checklist

Not all may apply:

• Tests (and test files) have been updated
• Documentation has been updated
  • If related to resstock-estimation, checklist includes data dictionary, source report, options_lookup.
  • If changes to project_testing tsvs, checklist includes yml_precomputed, yml_precomputed_outdated, yml_precomputed_weight
• Changelog has been updated
• openstudio tasks.rb update_measures has been run
• No unexpected regression test changes on CI (checked comparison artifacts)