-
Notifications
You must be signed in to change notification settings - Fork 183
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
Opengl rasterization for union #3538
Conversation
…on for shapes for brute-force rasterization.
@@ -66,6 +67,14 @@ class Union : public Shape { | |||
dist_compare); | |||
} | |||
|
|||
bool is_inside(Utils::Vector3d const &pos) const override { |
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.
@fweik this override is necessary if the union shape is used in a LB boundary.
src/core/shapes/Union.hpp
Outdated
@@ -66,6 +67,14 @@ class Union : public Shape { | |||
dist_compare); | |||
} | |||
|
|||
bool is_inside(Utils::Vector3d const &pos) const override { | |||
return boost::accumulate( |
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.
I think you should use any_of
, which will short-cut and makes the intent clearer.
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.
you probably mean all_of
9d3f13a
to
07940a5
Compare
07940a5
to
7279340
Compare
Codecov Report
@@ Coverage Diff @@
## python #3538 +/- ##
======================================
- Coverage 87% 87% -1%
======================================
Files 536 536
Lines 24180 24146 -34
======================================
- Hits 21066 21014 -52
- Misses 3114 3132 +18
Continue to review full report at Codecov.
|
i will provide a test for LB boundary using the union shape which will make codecov happy |
No description provided.