llnl / llnl/axom

Face orientation convention for structured meshes is wrong

Open
#52 0 comments 0 reactions 1 assignee Claimed by @gzagaris View on GitHub
bug Mint
Dominant language
C++
Stars
196
Forks
34
Avg merge
4d 1h
Merged PRs (30d)
11

Description

A mesh face can have at most two adjacent cells, C1 and C2. By convention the face is oriented according to an outward facing normal with respect to C1.

However, for structured meshes this convention is not enforced. For example, in 2D:
- All ``I``-Faces normals are pointing to the ``(+I)`` direction
- All ``J``-Faces normals are pointing to the ``(-J)`` direction

This can be problematic for operations that rely on the convention.

This should be relatively straight-forward to fix, by carefully handling the left-most boundary.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.