-
Notifications
You must be signed in to change notification settings - Fork 193
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
v24.1.0-IOFreeze: PeopleDefinition's EnclosureAveraged key #5100
Conversation
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.
Remove ZoneAverage completely. Change the PeopleDefinition::setMeanRadiantTemperatureCalculationType to replace ZoneAveraged to EnclosureAveraged
@joseph-robertson I made some changes, assuming the build works and tests don't look too broken, let's merge that back into the main IOFreeze branch. |
CI Results for 45185c5:
|
Pull request overview
For OS:People:Definition's "Mean Radiant Temperature Calculation Type" field, support key change from "ZoneAveraged" to "EnclosureAveraged".
Questions:
Pull Request Author
src/model/test
)src/energyplus/Test
)src/osversion/VersionTranslator.cpp
)Labels:
IDDChange
APIChange
Pull Request - Ready for CI
so that CI builds your PRReview Checklist
This will not be exhaustively relevant to every PR.