-
Notifications
You must be signed in to change notification settings - Fork 157
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
feat: add export to json geometry using hierarchy map #679
feat: add export to json geometry using hierarchy map #679
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.
Looks good to me, just one comment concerning the boundary.
// This is a navigation volume, write the boundaries | ||
if (writeBoundary) { | ||
for (auto bsurface : volume.boundarySurfaces()) { | ||
// ; |
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.
Is it normal that the writing of the boundary does nothing ?
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.
That's a place-holder, but I think I can add it. For writing the boundaries, I should actually resolve also the volume pointing, but then this would be a different type ... I will add "just" the plain surface information here.
Codecov Report
@@ Coverage Diff @@
## master #679 +/- ##
=======================================
Coverage 49.07% 49.07%
=======================================
Files 331 331
Lines 16568 16568
Branches 7722 7722
=======================================
Hits 8130 8130
Misses 3006 3006
Partials 5432 5432 Continue to review full report at Codecov.
|
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.
Addressing PR comments.
Merging master in & then have a look. |
Adding Surface writing capability to the framework.
This PR adds a Geometry/Surface converter for the
Geometry
examples in the framework, using theGeometryHierarchyMapConverter<>
from the Core repository.It also corrects a faulty include directory from
AtlasExamples/Plugin
toAtlasExamples/Io
.It needed also a
to_json
conversion forstd::shared_ptr<const Surface> surface
rather than theconst Surface&
.