Skip to content
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

Euclidean shapes csg #962

Merged
merged 9 commits into from Mar 21, 2015
Merged

Conversation

jlevallois
Copy link
Member

Refactoring of CSG operations on Euclidean / Digital shapes.
Can combine easily several operations.

EuclideanShapesUnion, EuclideanShapesIntersection and EuclideanShapesMinus are now deprecated. Use EuclideanShapesCSG instead.
DigitalShapesUnion, DigitalShapesIntersection and DigitalShapesMinus are now deprecated. Use DigitalShapesCSG instead.

golf

:octocat:

@@ -49,6 +49,16 @@
ConstIterator and adds a new method to change the color of a
specific face. (Bertrand Kerautret,
[#937](https://github.com/DGtal-team/DGtal/pull/937))
- Refactoring of CSG operations on Euclidean / Digital shapes.
Can combine easily several operations.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refactoring... to easily combine several operations?

@dcoeurjo
Copy link
Member

Very nice. Could you please update the module shape documentation ?

:octocat:

@jlevallois
Copy link
Member Author

ModuleShape documentation was updated since exampleEuclideanShapeDecorator was updated too (snipet code). But, yes, I will take advantage of this PR to improve the documentation.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 90.35% when pulling b1388d0 on jlevallois:EuclideanShapesCSG into bbf8b31 on DGtal-team:master.

@@ -54,6 +54,15 @@
[#937](https://github.com/DGtal-team/DGtal/pull/937))
- New methods to generate basic 3D tubular meshes and height fields. New mesh module documentation added.
(Bertrand Kerautret, [#969](https://github.com/DGtal-team/DGtal/pull/969))
- Refactoring of CSG operations on Euclidean / Digital shapes to easily
combine easily several operations.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"easily" twice

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's really easy.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sense....

Jeremy Levallois added 2 commits March 20, 2015 17:45
@dcoeurjo
Copy link
Member

Thanks merging

dcoeurjo added a commit that referenced this pull request Mar 21, 2015
@dcoeurjo dcoeurjo merged commit 2086e91 into DGtal-team:master Mar 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants