<table><tr><td style="">dkolozsvari updated this revision to Diff 6090.<br />dkolozsvari added a comment.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2444" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>This revision solves the issue mentioned with the bathymetry polygons missing. The problem was with the orientation of the inner rings: they are counterclockwise polygons, but the clipping algorith works ONLY on clockwise polygons, so implementing an efficient way to reverse the order of the nodes solved the problem.</p>

<p>Besides that, I had to track down another issue in the BaseClipper.cpp, because I noticed, that some inner rings are not even added to the files. The clipping algorithm with the TinyPlanetProcessor class works fine now, landmass polygons and bathymetry polygons too are parsed, cut and outputted correctly.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rMARBLE Marble</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D2444?vs=5915&id=6090" rel="noreferrer">https://phabricator.kde.org/D2444?vs=5915&id=6090</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2444" rel="noreferrer">https://phabricator.kde.org/D2444</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>tools/osm-simplify/BaseClipper.cpp<br />
tools/osm-simplify/BaseClipper.h<br />
tools/osm-simplify/TinyPlanetProcessor.cpp<br />
tools/osm-simplify/TinyPlanetProcessor.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>dkolozsvari, nienhueser, rahn<br /><strong>Cc: </strong>Marble, shentey, chaz6, dkolozsvari, cmihalache, rahn, nienhueser, marble-devel<br /></div>