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

Unit test to demonstrate #15 #29

Merged
merged 3 commits into from Nov 12, 2016
Merged

Unit test to demonstrate #15 #29

merged 3 commits into from Nov 12, 2016

Conversation

madhephaestus
Copy link
Contributor

I wrote a stripped down unit test to demonstrate the issue i have been describing in #15 . This unit test will test to see that when the extrude function is called on convex polygons, that the triangulation function does not fall off the edge and begin placing triangles inside out.

hephaestus added 3 commits November 9, 2016 18:08
This unit test is for testing concave polygons with the extrusion
feature.
In this unit test I create 2 concave polygons. These polygon data is
derrived from a path iterator used to create fonts. The 2 letters chosen
are a capital H and lower case e (just the outer shape, the inner hole
has been removed from this dataset to demonstrate a specific problem).
The 2 sets of points are extruded and the number of polygons displayed
of the 2. When a simple union of non intersecting CSG's is performed,
the total number of polygons should not change.
@madhephaestus
Copy link
Contributor Author

This will fail the travis build because it includes a unit test that demonstrated a fault and at this time is not passing.

@miho
Copy link
Owner

miho commented Nov 12, 2016

Thanks for preparing the test. I'll have a look at it.

@miho miho merged commit bff50ba into miho:master Nov 12, 2016
@miho
Copy link
Owner

miho commented Nov 12, 2016

that the triangulation function does not fall off the edge and begin placing triangles inside out.

Your assumption in the test is wrong. The increased number of triangles does not come from bad orientation etc.

Just try this with regular primitives such as two spheres and it will fail as well if you don't enable optimization.

@madhephaestus
Copy link
Contributor Author

How would i enable optimizations?

@miho
Copy link
Owner

miho commented Nov 12, 2016

See your unit test. I modified it to work properly. But I doubt that this
is the root of your problem.

Dipl.-Math. Michael Hoffer

Twitter: @mihosoft
Webpage: www.mihosoft.eu

Goethe-Zentrum für Wissenschaftliches Rechnen (G-CSC)
Goethe-Universität
Kettenhofweg 139
60325 Frankfurt am Main
phone: +49 69 798 25254
info@michaelhoffer.de

2016-11-12 20:42 GMT+01:00 Kevin Harrington notifications@github.com:

How would i enable optimizations?


You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#29 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AANv8nzP3E4aDIJb4v5VMBZdhyX-gKURks5q9haOgaJpZM4KwdrA
.

@madhephaestus
Copy link
Contributor Author

Why is this mode not the default?

@miho
Copy link
Owner

miho commented Nov 13, 2016

Because it does not work 100% reliable. There are edge cases that I didn't test so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants