Skip to content
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

Vertex coordinate order changed after cull #602

Open
changliao1025 opened this issue Feb 4, 2025 · 4 comments
Open

Vertex coordinate order changed after cull #602

changliao1025 opened this issue Feb 4, 2025 · 4 comments

Comments

@changliao1025
Copy link

After the cull operation, some mesh cell vertex coordinates are switched in order, which creates invalid polygons.
Examples: using both the base and culled mesh, we can plot the mesh cell using the same cell center location:

Image
Image

and

Image
Image

The test mesh files are stored at:
/compyfs/liao313/mpas/out

@changliao1025
Copy link
Author

@xylar I shared a minimal reproducible code at: https://gist.github.com/changliao1025/939376071ead9dfc89ccc0ce27b04ed9

@xylar
Copy link
Collaborator

xylar commented Feb 4, 2025

@changliao1025, thanks for reporting this problem. Sorry that it's happening!

Just to be sure I understand, is the base mesh before or after it has gone through the mesh converter? (After, I assume, so we can eliminate that that is responsible?)

@changliao1025
Copy link
Author

changliao1025 commented Feb 4, 2025

@changliao1025, thanks for reporting this problem. Sorry that it's happening!

Just to be sure I understand, is the base mesh before or after it has gone through the mesh converter? (After, I assume, so we can eliminate that that is responsible?)

The base mesh was produced using the mesh converter, so it is not the JIGSAW triangular.

@changliao1025
Copy link
Author

There are a total of 189 incorrect cells in the mesh, and they are (possibly) all located on the coastline:

Image

For example:

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants