https://code.cor-lab.de/https://code.cor-lab.de/favicon.ico?14019720732015-03-31T14:39:25ZOpen Source Collaboration PlatformRobotics Systems Types - Feature #2211: Add PlanePatch to the sandboxhttps://code.cor-lab.de/issues/2211?journal_id=67182015-03-31T14:39:25ZJ. Moringenjmoringe@cor-lab.uni-bielefeld.de
<ul><li><strong>File</strong> <a href="/attachments/download/467/0001-Added-new-type-PolygonalPatch3D-to-sandbox.patch">0001-Added-new-type-PolygonalPatch3D-to-sandbox.patch</a><a href="/attachments/467/0001-Added-new-type-PolygonalPatch3D-to-sandbox.patch"><img alt="Magnifier" src="/images/magnifier.png?1396112202" /></a> added</li></ul><p>Suggested changes are attached. Please comment (especially on the TODOs).</p> Robotics Systems Types - Feature #2211: Add PlanePatch to the sandboxhttps://code.cor-lab.de/issues/2211?journal_id=67192015-03-31T15:09:53ZL. Zieglerlziegler@techfak.uni-bielefeld.de
<ul></ul><p>Those changes make sense, I agree.</p>
<p>What do you mean by "self intersection"? Is this related to the polygon?<br />Personally, I would not expect self intersection within the polygon.<br />But I don't think the polygon should be forced to be convex.</p>
<p>I think the most intuitive and winding rule would be ccw w.r.t. normal (right hand rule).</p>
<p>Duplicate entries make no sense, but is it necessary to explicitly mention this here?</p> Robotics Systems Types - Feature #2211: Add PlanePatch to the sandboxhttps://code.cor-lab.de/issues/2211?journal_id=67292015-04-07T08:33:30ZL. Zieglerlziegler@techfak.uni-bielefeld.de
<ul><li><strong>File</strong> <a href="/attachments/download/468/0001-Add-new-type-PolygonalPatch3D-to-sandbox.patch">0001-Add-new-type-PolygonalPatch3D-to-sandbox.patch</a><a href="/attachments/468/0001-Add-new-type-PolygonalPatch3D-to-sandbox.patch"><img alt="Magnifier" src="/images/magnifier.png?1396112202" /></a> added</li></ul><p>Attached version with resolved TODOs.</p> Robotics Systems Types - Feature #2211: Add PlanePatch to the sandboxhttps://code.cor-lab.de/issues/2211?journal_id=67302015-04-07T10:08:16ZL. Zieglerlziegler@techfak.uni-bielefeld.de
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a href="https://code.cor-lab.de/projects/rst/repository/revisions/ca86176cb59aacbf6a7157cb13bb8ef8614fe833" class="changeset" title="Add new type PolygonalPatch3D to sandbox fixes #2211 Signed-off-by: Johannes Wienke <jwienke@te...">rst-proto|ca86176cb59aacbf6a7157cb13bb8ef8614fe833</a>.</p>