I may have jumped the gun on merging this ResStockArgumentsPostHPXML refactor PR from post_hpxml_class into latest_os_hpxml without first waiting for @rajeee 's review. The code changes are the same as the original closed/merged PR by @shorowit , and the description (below) is the same as well. I provided my review to Scott's changes (see original closed/merged PR #1388 ) and they look good to me, but it would good to have Rajendra's review as well in case we missed anything, hence "reopening" the PR.

Pull Request Description

Noticed this while working with @Sashadf1 on #1351.

Refactors the ResStockArgumentsPostHPXML measure to use the HPXML class rather than direct XML manipulation. Probably direct XML manipulation was used because it copied the approach used in the OS-HPXML BuildResidentialScheduleFile measure, but there's really no need for it. It's simpler/cleaner to use the HPXML class. (I'm refactoring the BuildResidentialScheduleFile measure here to also use the HPXML class.)

Note that I didn't do any testing beyond checking that the unit tests all pass. I assume that's sufficient?

Checklist

Required:

• Add to the changelog_dev.rst file
• No unexpected regression test changes on CI (comparison artifacts have been checked)
• Update documentation (one is required)
  • Technical reference guide
  • Technical development guide

Optional (not all items may apply):

• Tests (and test files) have been updated
• Supporting resource files have been updated (related to resstock-estimation)
  • data dictionary
  • source report
  • options saturation
  • options_lookup
• openstudio tasks.rb update_measures has been run