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.

Sign up for GitHub

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

Jump to bottom

IJK vs Ensight #124

Open

kpwelsh opened this issue Nov 1, 2023 · 0 comments

Collaborator

kpwelsh commented Nov 1, 2023 •

edited

Loading

Some parts of the code use IJK and some use the EnSight node ordering convention.

It might not technically be a problem right now, but its bound to cause us issues in the future if we don't sort that out.

We should have type guarded connectivity structures to prevent us from ever making that issue in the future.

Could also just always use the same convention internally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment