-
Notifications
You must be signed in to change notification settings - Fork 68
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
Fix vcs3d failing test #1305
Comments
@aashish24 if this is the same issue as https://open.cdash.org/testDetails.php?test=334149793&build=3808629 - it's DVD3D not VCS3D that's got the problem. It looks to me like it's a case of continental bounds missing a single point, so the start not returning to the exact end point, closing the polygon. |
Agreed -- shouldn't the DV3D team be looking at this? |
The code that generates the boundaries is in shapeFileReader.getPolyLines in Packages/DV3D/Shapefile.py. — Tom From: "Paul J. Durack" <[email protected]mailto:[email protected]> @aashish24https://github.com/aashish24 if this is the same issue as https://open.cdash.org/testDetails.php?test=334149793&build=3808629 - it's DVD3D not VCS3D that's got the problem. It looks to me like it's a case of continental bounds missing a single point, so the start not returning to the exact end point, closing the polygon. — |
If I'm reading the file that @ThomasMaxwell pointed at correctly, the continents are defined in a GIS shapefile, which is then rendered using |
@dlonie Looks like the shapefile is converted to VTK here: |
@dlonie @ThomasMaxwell is busy with other tasks and we discussed that vcs3D (visualization related) changes will be taken care by you and me (obviously with @ThomasMaxwell's help). |
and @chaosphere2112 we also discussed that @aashish24 and @dlonie are very busy and wouldn't want you guys charging your precious time on things we an also fix 😉 |
@doutriaux1 we are never too busy for you 👍 |
Awwww... You're too nice 😉 Anyhow you do run out of money though 😜 |
I think we all do -:) |
No description provided.
The text was updated successfully, but these errors were encountered: