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
Fix#127 - Fix volume calculation to match EnergyPlus #4592
Conversation
…al to another, and if a point is on a line segment Matches E+
…d calcualte volume correctly (like E+ does)
the windows build error makes no sense.
|
CI Results for 47d37aa:
|
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 is great to have this issue resolved! many years.
Pull request overview
isEnclosedVolume()
which check that each edge/segment is used twice exactly, potentially splitting up existing edges (this is similar to what intersectSurfaces would do, but does not reuse that portion)Space::volume is enhanced to do:
Pull Request Author
src/model/test
)Labels:
IDDChange
APIChange
Pull Request - Ready for CI
so that CI builds your PRReview Checklist
This will not be exhaustively relevant to every PR.