Feature #2211
Add PlanePatch to the sandbox
Status: | Resolved | Start date: | 03/31/2015 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 100% | |
Category: | Type Proposal | |||
Target version: | - |
Description
Please add the new type to the sandbox and back-port to 0.11. Thanks!
Associated revisions
Add new type PolygonalPatch3D to sandbox
fixes #2211
Signed-off-by: Johannes Wienke <jwienke@techfak.uni-bielefeld.de>
Add new type PolygonalPatch3D to sandbox
fixes #2211
Signed-off-by: Johannes Wienke <jwienke@techfak.uni-bielefeld.de>
(cherry picked from commit ca86176cb59aacbf6a7157cb13bb8ef8614fe833)
History
#1 Updated by J. Moringen almost 9 years ago
Suggested changes are attached. Please comment (especially on the TODOs).
#2 Updated by L. Ziegler almost 9 years ago
Those changes make sense, I agree.
What do you mean by "self intersection"? Is this related to the polygon?
Personally, I would not expect self intersection within the polygon.
But I don't think the polygon should be forced to be convex.
I think the most intuitive and winding rule would be ccw w.r.t. normal (right hand rule).
Duplicate entries make no sense, but is it necessary to explicitly mention this here?
#3 Updated by L. Ziegler almost 9 years ago
Attached version with resolved TODOs.
#4 Updated by L. Ziegler almost 9 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 100
Applied in changeset rst-proto|ca86176cb59aacbf6a7157cb13bb8ef8614fe833.