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

Fix assignment of bedrooms to units #330

Merged
merged 2 commits into from
Oct 17, 2019
Merged

Fix assignment of bedrooms to units #330

merged 2 commits into from
Oct 17, 2019

Conversation

joseph-robertson
Copy link
Contributor

Pull Request Description

This fixes a bug when you try to assign, e.g., 3, 2, 1, 2, 2, 3 numbers of bedrooms to 6 building units.

Checklist

Not all may apply:

  • Unit tests have been added or updated
  • The update_measures rake task has been run
  • The test:regenerate_osms rake task has been run
  • Unit tests and integrity checks all pass locally
  • PAT project measures/outputs have been updated
  • Documentation has been modified appropriately
  • Any new options are added to project_testing
  • project_testing runs without any failures
  • No unexpected circleci regression test changes
  • All tests are passing (green) on circleci
  • This branch is up-to-date with master

For more information on how to perform these checklist items, see the documentation's Advanced Tutorial.

@joseph-robertson joseph-robertson self-assigned this Oct 16, 2019
@joseph-robertson joseph-robertson merged commit 7a44d4b into master Oct 17, 2019
@joseph-robertson joseph-robertson deleted the unit-bdrms branch October 17, 2019 14:31
@joseph-robertson joseph-robertson added this to the ResStock v2.1.0 milestone Jan 30, 2020
joseph-robertson added a commit that referenced this pull request Mar 26, 2020
0a562998c5 Update test osw files.
cd0aced18a Remove a test comparison exclusion.
c7655c2936 Update some test osw files.
4ea1b02cd0 Start to make some arguments optional.
75930cdec7 Regenerate test osw files.
94ec655b90 Expose water heater setpoint temp and hp backup switchover temp.
b249fb040a Run update_measures.
496b4a1435 Update build measure and test osws.
a5400892bd Merge branch 'master' into build-res-hpxml
f8c20fee25 Merge pull request #331 from NREL/model-reset
12a184406c Update warning message. [ci skip]
140aff3932 Remove any existing objects from the model.
72b4b997d4 Merge pull request #328 from NREL/init_defaults
f67b996964 Various code cleanup.
a3452ed71a Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML into init_defaults
7dc076a7b6 Merge pull request #330 from NREL/build-res-hpxml-compare-xml
66155c08a7 Removes compare-xml gem, replaced with simple string comparisons.
5480f9b83d Update translator measure to remove existing model objects.
9e4058fb7a Bugfix for HEScore.
545d47d50b Update tests.
5d02ea72ee Include more defaults.
ff69e52994 First pass on setting HPXML defaults up front. Updates measure arguments to support additional output products. Adds support for writing the HPXML file (after defaults applied).
8e44fbbcbb Merge pull request #321 from NREL/operable_windows
190b7d15b3 Update number of residents.
6778aae5b6 More updates to test files.
5822021f78 Update some test file default values.
6573376398 Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML into operable_windows
c7741eb61f Fix test.
4bb3370e38 Bugfix.
448af9fe28 Bugfixes.
1a16300329 Clean up some test files.
e163da8cc2 Fix tests.
03c31596fb Further reduce test files. Adds a single base-misc-defaults.xml to cover most defaults.
4922a04314 Reduce number of test files.
2b010da0f4 Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML into operable_windows
4b92755558 Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML into operable_windows
f5f53a169f Minor improvements to get_default_azimuths.
89b64fe2db Bugfix.
dd3eb1fe08 Moves calculation to HPXML class.
a56732fb2b Switches from BuildingConstruction/extension/FractionofOperableWindowArea to Window/Operable while preserving the minimum number of unique windows modeled in EnergyPlus for reasons of speed.

git-subtree-dir: resources/hpxml-measures
git-subtree-split: 0a562998c5a1d9bd857387485e352a2f24ab09f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant