-
Notifications
You must be signed in to change notification settings - Fork 50
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
First step of QuadMesh #595
Conversation
* and some spacing. * triangulate using sewing.
Since clang-format is now added to git repo, do not create a symlink.
Solve test depends on uninit value (displayed by valgrind).
As kitten died when I copy paste this code ... without changing the test.
Replace TriangleMeshComponent by a template class SurfaceMeshComponent Define TriangleMeshComponent as SurfaceMeshComponent<TriangleMesh> Update SurfaceMeshComponent to support Polygonal Meshes.
Indices are used by setFaces.
Here is an overview of what got changed by this pull request: Issues
======
- Added 4
Clones removed
==============
+ src/Engine/Component/GeometryComponent.cpp -6
See the complete overview on Codacy |
auto hepta = VectorNui( 7 ); | ||
hepta << 3, 2, 4, 5, 6, 7, 8; | ||
auto degen = VectorNui( 10 ); | ||
degen << 1, 0, 9, 10, 11, 12, 13, 14, 15, 16; |
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.
Codacy found an issue: Found suspicious operator ','
polyMesh.setNormals( normals ); | ||
|
||
auto quad = VectorNui( 4 ); | ||
quad << 0, 1, 2, 3; |
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.
Codacy found an issue: Found suspicious operator ','
auto degen = VectorNui( 10 ); | ||
degen << 1, 0, 9, 10, 11, 12, 13, 14, 15, 16; | ||
auto degen2 = VectorNui( 10 ); | ||
degen2 << 14, 13, 12, 11, 10, 9, 17, 18, 16, 15; |
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.
Codacy found an issue: Found suspicious operator ','
auto quad = VectorNui( 4 ); | ||
quad << 0, 1, 2, 3; | ||
auto hepta = VectorNui( 7 ); | ||
hepta << 3, 2, 4, 5, 6, 7, 8; |
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.
Codacy found an issue: Found suspicious operator ','
This PR corresponds to the addition of Core and Engine QuadMesh.
Tological mesh is not yet supported.
See #589 for discussion and details.
here from top to bottom, heptagone, quad, and two degenerated (align points, equal points) decagon
PolyMesh
when required