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 #4538, wrap phase change material properties #4610
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.
Jumping in unannounced because I saw swig build error. I didn't do a full review since this is still marked as a draft.
@jmarrec I responded to most of your comments. What (optionally?) remains:
|
src/energyplus/ForwardTranslator/ForwardTranslateDaylightRedirectionDevice.cpp
Outdated
Show resolved
Hide resolved
src/energyplus/ForwardTranslator/ForwardTranslateRoofVegetation.cpp
Outdated
Show resolved
Hide resolved
src/energyplus/ForwardTranslator/ForwardTranslateStandardOpaqueMaterial.cpp
Show resolved
Hide resolved
src/energyplus/ForwardTranslator/ForwardTranslateMasslessOpaqueMaterial.cpp
Show resolved
Hide resolved
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.
Suggested an IDD change (which we should probably propose to E+ on the issue board too), otherwise I think it's ready to drop! Thanks for the new objects @joseph-robertson !
CI Results for e9677bc:
|
Pull request overview
MaterialProperty:PhaseChange
MaterialProperty:PhaseChangeHysteresis
MaterialPropertyMoisturePenetrationDepthSettings
.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.