-
Notifications
You must be signed in to change notification settings - Fork 56
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
V2 : reTesselateCoplanarPolygons
seems to have a problem
#194
Comments
reTesselateCoplanarPolygons
seems to have a problemreTesselateCoplanarPolygons
seems to have a problem
@FishOrBear this is really good. I’m not too surprised as that code needs some rework, and the transition to V2 was not straight forward. Anyway, I’ll take a look at this. Do you want to follow along? If so then I can post updates. |
Can this result be without overlapping triangles? how do you want to do it |
@FishOrBear you seem to have specific requirements for the results. Are there errors in the polygons after this routine finishes? |
Extracting the edges of a 3D geometry would be something very special. In general, the goal of reTesselate is to combine small polygons. The CSG boolean operations tend to chop polygons into smaller and smaller polygons. Therefore, reTesselate was born to combine co-planar polygons that share opposing edges. But the result will never be anything close to the 'edges'. |
Well, I have another algorithm to extract it. So I temporarily closed |
You have sad that twice now.... ‘triangles overlap’. Can you provide two overlapping triangles from the example above? I’m asking because I rewrote reTesselate. The new algorithm does not create any new 3D vertices, so overlapping will not be possible. But I’d like to have a test case. |
I think this use case will work
|
@FishOrBear thanks small comment... 3D polygons contain order sets of unique points. the data provided has an redundant first and last point. |
Here's the result of the new reTesselate function. Only 27 polygons. 😃
|
This result is very good, can I experience it on the current v2 branch? update: |
The new function is still under testing. Please wait a few more days. |
@z3dev OK, look forward to your good news. |
reTesselateCoplanarPolygons
seems to have a problemreTesselateCoplanarPolygons
seems to have a problem
ver:V2
Primitive triangle mesh
reTesselateCoplanarPolygons (draw line
reTesselateCoplanarPolygons (draw face
testcode:
Is this expected?
v0.7 testcode:
v0.7 dest count:35
v2 dest count:54
Is there an overlap in the generated polygon, is this also expected?
The text was updated successfully, but these errors were encountered: