Feature #2211

Add PlanePatch to the sandbox

Added by L. Ziegler almost 6 years ago. Updated almost 6 years ago.

Status:ResolvedStart date:03/31/2015
Priority:NormalDue 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!

0001-Added-new-type-PlanePatch-to-sandbox.patch Magnifier (1.88 KB) L. Ziegler, 03/31/2015 03:24 PM

0001-Added-new-type-PolygonalPatch3D-to-sandbox.patch Magnifier (2.49 KB) J. Moringen, 03/31/2015 04:38 PM

0001-Add-new-type-PolygonalPatch3D-to-sandbox.patch Magnifier (2.38 KB) L. Ziegler, 04/07/2015 10:32 AM

Associated revisions

Revision ca86176c
Added by L. Ziegler almost 6 years ago

Add new type PolygonalPatch3D to sandbox

fixes #2211

Signed-off-by: Johannes Wienke <>

Revision 106eb44b
Added by L. Ziegler almost 6 years ago

Add new type PolygonalPatch3D to sandbox

fixes #2211

Signed-off-by: Johannes Wienke <>
(cherry picked from commit ca86176cb59aacbf6a7157cb13bb8ef8614fe833)

History

#1 Updated by J. Moringen almost 6 years ago

Suggested changes are attached. Please comment (especially on the TODOs).

#2 Updated by L. Ziegler almost 6 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 6 years ago

Attached version with resolved TODOs.

#4 Updated by L. Ziegler almost 6 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF