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
Addresses #4531, breaking changes #4548
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.
No problem with these changes. I'd like a quick one or two line tests for the PlantLoop method probably.
And I will run a full OpenStudio-Resources test suite to check whether we need API version switches over there.
OS-resources run at https://github.com/NREL/OpenStudio-resources/runs/5540792038?check_suite_focus=true (in progress) |
OS-resources run is clean, no new failures. |
CI Results for 591f144:
|
Pull request overview
evaporationLossMode
(boost::optional<std::string>
tostd::string
)blowdownCalculationMode
(boost::optional<std::string>
tostd::string
)cellControl
for CoolingTower:VariableSpeedcommonPipeSimulation
(boost::optional<std::string>
tostd::string
)isCommonPipeSimulationDefaulted
commonPipeSimulationValues
gettervalidCommonPipeSimulationValues
getterPull 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.