Non-transient dwelling unit energy recovery ventilation #1165
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yunyangye
added
the
90.1-2016/9
yunyangye
changed the title
yunyangye
changed the title
lymereJ
reviewed
Aug 24, 2021
lib/openstudio-standards/prototypes/ashrae_90_1/ashrae_90_1_2019/ashrae_90_1_2019.Model.rb
Outdated
Show resolved
Hide resolved
…ototype dwelling units whether there is a need for energy recovery or not, if the latter, the ERV only provides ventilation air. Ventilation air at the system is shut-off and the system cycles to meet the load only. The system ERV if added is removed but is used as an indication if energy recovery is needed. Effectiveness of the residential ERV is set based the Enthalpy Recovery Ratio (ERR), which is translated to effectiveness based on regression obtained by analyizing manufacturer data. The translation is design condition dependent (heating or cooling), this corresponds to the err_basis in the code.
mdahlhausen
reviewed
Sep 13, 2021
| @@ -98,6 +98,8 @@ def unique_properties(sheet_name) | |||
| ['template', 'fluid_type', 'fuel_type', 'condensing', 'condensing_control', 'minimum_capacity', 'maximum_capacity', 'start_date', 'end_date'] | |||
| when 'chillers' | |||
| ['template', 'cooling_type', 'condenser_type', 'compressor_type', 'absorption_type', 'variable_speed_drive', 'minimum_capacity', 'maximum_capacity', 'start_date', 'end_date'] | |||
| when 'furnaces' | |||
| ['template', 'minimum_capacity', 'maximum_capacity', 'start_date', 'end_date'] | |||
There was a problem hiding this comment.
merge in the other commit for furnaces to this pull request
mdahlhausen
reviewed
Sep 13, 2021
| @@ -26,6 +26,32 @@ | |||
| 1.0 | |||
| ] | |||
| }, | |||
| { | |||
| "name": "Always On - No DD", | |||
There was a problem hiding this comment.
Use full name 'Design Day' instead of DD
mdahlhausen
reviewed
Sep 13, 2021
| ] | ||
| }, | ||
| { | ||
| "name": "Always On - No DD", |
There was a problem hiding this comment.
'Design Day' instead of 'DD'
mdahlhausen
reviewed
Sep 13, 2021
| "10_to_20_percent_oa": null, | ||
| "20_to_30_percent_oa": null, | ||
| "30_to_40_percent_oa": 5500.0, | ||
| "40_to_50_percent_oa": 4500.0, | ||
| "50_to_60_percent_oa": 3500.0, | ||
| "60_to_70_percent_oa": 2000.0, | ||
| "70_to_80_percent_oa": 1000.0, | ||
| "greater_than_80_percent_oa": 0.0 | ||
| "greater_than_80_percent_oa": 0.0, | ||
| "err_basis": null, |
There was a problem hiding this comment.
use 'enthaply_recovery_ratio' instead of 'err'
mdahlhausen
reviewed
Sep 13, 2021
| # Regressions were determined based available manufacturer data. | ||
| # | ||
| # @param err [float] Enthalpy Recovery Ratio (ERR) | ||
| # @param basis [String] basis for effectiveness calculation, either cooling or heating |
There was a problem hiding this comment.
'cooling' or 'heating'
mdahlhausen
reviewed
Sep 13, 2021
| # | ||
| # This function is only used for nontransient dwelling units (Mid-rise and High-rise Apartment) | ||
| # @param [OpenStudio::Model::HeatExchangerAirToAirSensibleAndLatent] heat exchanger air to air sensible and latent | ||
| # @param [String] err |
There was a problem hiding this comment.
parameters not fully explained / specified
mdahlhausen
reviewed
Sep 13, 2021
| # @param [String] err | ||
| # @param [String] err basis (Cooling/Heating) | ||
| # @param [String] climate zone | ||
| def heat_exchanger_air_to_air_sensible_and_latent_apply_prototype_efficiency_err(heat_exchanger_air_to_air_sensible_and_latent, err, basis, climate_zone) |
There was a problem hiding this comment.
even though the method name is long, avoid acronyms ('err') in method names
mdahlhausen
requested changes
Sep 13, 2021
…ype from SAC to PSZ-AC to be able to model cycling fan.
|
@mdahlhausen - Just pushed a commit that addresses the review comments. Please note that there's an additional change: the type of system used by the midrise apartment model. It was original setup to use a split system AC but I ran into #1208 when wanting to change the constant volume fan to cycling. Using a PSZ-AC instead allow the fan to cycle to meet the load which is what we want: system to cycle to meet the load and ERV to provide constant OA with or without energy recovery. |
mdahlhausen
approved these changes
Sep 14, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the implementing method about ERV for non-transient dwelling units. The energy simulations require inputs in terms of heat recovery effectiveness. This PR translates the ERR values to effectiveness and insert the new effectiveness into zone ERV. This implementation affects the prototype models for 90.1-2019 mid-rise and high-rise apartments. Additional information regarding the implementation are provided in Section B2.3 in the Energy Savings Analysis: ANSI/ASHRAE/IES Standard 90.1-2019 report.