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

Remove limits for "Minimum Outdoor Drybulb Temperature for Compressor Operation" field in DX Heating Coil objects #6190

Closed
3 tasks
mjwitte opened this issue Jun 29, 2017 · 1 comment
Assignees
Labels
PriorityHigh This defect or feature has been declared high priority

Comments

@mjwitte
Copy link
Contributor

mjwitte commented Jun 29, 2017

Issue overview

All DX heating coil objects (space heating and water-heating) have an input field for "Minimum Outdoor Drybulb Temperature for Compressor Operation". Most of them have a minimum specified in the IDD, while some do not. Specifically, the limit was removed for multi-speed heat pumps in #4383.

The minimum limit should be removed from the other DX heating coils, space heating coils for sure, and also water heating coils?

At a "minimum" :-) this requires IDD changes, but there may also be checks in the code that need to be removed. And maybe some doc changes as well.

ZoneHVAC:PackagedTerminalHeatPump,
   N8 , \field Minimum Outdoor Dry-Bulb Temperature for Compressor Operation
        \type real
        \minimum -20.0

Coil:Heating:DX:SingleSpeed,
  N5 , \field Minimum Outdoor Dry-Bulb Temperature for Compressor Operation
       \type real
       \minimum -20.0

Coil:Heating:DX:VariableSpeed,
   N5,  \field Minimum Outdoor Dry-Bulb Temperature for Compressor Operation
        \type real
        \minimum -50.0

WaterHeater:HeatPump:PumpedCondenser,
  N4 , \field Minimum Inlet Air Temperature for Compressor Operation
       \type real
       \units C
       \minimum -5

WaterHeater:HeatPump:WrappedCondenser,
  N5 , \field Minimum Inlet Air Temperature for Compressor Operation
       \type real
       \units C
       \minimum -5

Details

Some additional details for this issue (if relevant):

  • Platform (Operating system, version)
  • Version of EnergyPlus v8.7
  • Unmethours link or helpdesk ticket number

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.

  • No defect files - should modify pertinent example files and test with a low limit.
  • Ticket added to Pivotal for defect (development team task)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
@mjwitte mjwitte added the PriorityHigh This defect or feature has been declared high priority label Jun 29, 2017
@Nigusse Nigusse self-assigned this Jul 18, 2017
Nigusse added a commit that referenced this issue Jul 18, 2017
Myoldmopar added a commit that referenced this issue Aug 7, 2017
Removed minimum limits of the minimum OA DB Temp for compressor operation input field #6190
@Myoldmopar
Copy link
Member

Closed via #6213

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PriorityHigh This defect or feature has been declared high priority
Projects
None yet
Development

No branches or pull requests

3 participants