-
Notifications
You must be signed in to change notification settings - Fork 57
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
Assign above-grade height for apartment units #1304
Conversation
It might be a minor impact, but I'd suggest defaulting to a 2ft belly for mobile homes. That's what OS-HPXML does (see here). |
Hmm maybe we were already defaulting to the 2 ft for Mobile Home w/ Ambient foundation type? |
After talking a bit with @joseph-robertson, my initial reaction is that the heating energy (natural gas) goes up large enough that I might suggest we assign some value to the argument based on height. I am a little hesitant is that our modeling of large MF buildings is sort of limited right now. |
@asparke2 : I wanted to let you know that we are considering using a new capability in OS-HPXML for multi-family units that are located above ground level. This would allow us to assign a height of the unit that impacts windspeed and resulting load due to infiltration. This is just to inform you of Multi-Family-related things coming down the pipeline as we plan the Multi-Family work. |
What about the space (height) between floors? We should look at what OS-HPXML does, and whether we should consider, e.g., plenums, etc. |
@joseph-robertson : The differences are relatively small and have diminishing returns as the height increases. Let's not include the distance between floors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the tests pass, let's merge this.
Pull Request Description
Closes #1303.
Proposal:
(n_floors - 1) * ceiling_height
(n_floors - 1) / 2 * ceiling_height
Update BuildExistingModel to register unit height above grade as
build_existing_model.unit_height_above_grade
.Fuel Use: Electricity: Total
Fuel Use: Natural Gas: Total
HVAC Design Load: Heating: Total
HVAC Design Load: Cooling Latent: Total
HVAC Design Load: Cooling Sensible: Total
Checklist
Not all may apply:
Tests (and test files) have been updatedIf related to resstock-estimation, checklist includes data dictionary, source report, options saturation, options_lookup.openstudio tasks.rb update_measures
has been run