-
Notifications
You must be signed in to change notification settings - Fork 377
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
Correct WSHP constant fan operating mode #10501
base: develop
Are you sure you want to change the base?
Conversation
state.dataUnitarySystems->CompOffMassFlow = this->m_HeatMassFlowRate[HeatSpeedNum]; | ||
state.dataUnitarySystems->CompOffFlowRatio = this->m_HeatMassFlowRate[HeatSpeedNum]; | ||
} else { | ||
state.dataUnitarySystems->CompOffMassFlow = this->m_HeatMassFlowRate[HeatSpeedNum - 1]; | ||
state.dataUnitarySystems->CompOffFlowRatio = this->m_MSHeatingSpeedRatio[HeatSpeedNum - 1]; |
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.
This was the problem in the defect file. In heating mode, when constant fan is used, the OFF mass flow rate should be the same as the ON mass flow rate.
EXPECT_EQ(1.2, state->dataUnitarySystems->CompOnMassFlow); | ||
EXPECT_EQ(0.6, state->dataHVACGlobal->MSHPMassFlowRateLow); | ||
EXPECT_EQ(1.2, state->dataHVACGlobal->MSHPMassFlowRateLow); | ||
EXPECT_EQ(1.2, state->dataHVACGlobal->MSHPMassFlowRateHigh); |
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.
This unit test should have noticed the issue with on and off air flow. This is correct now.
Top row is from PackagedTerminalAirConditionerVSAS and bottom row is PackagedTerminalHeatPumpVSAS example files that show big diffs. The fan/airflow is now constant as expected. The VS coils model constant air flow at speed which may not be the best choice for constant fan operating mode but that's what these coils do now. |
Pull request overview
NOTE: ENHANCEMENTS MUST FOLLOW A SUBMISSION PROCESS INCLUDING A FEATURE PROPOSAL AND DESIGN DOCUMENT PRIOR TO SUBMITTING CODE
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Reviewer
This will not be exhaustively relevant to every PR.