Skip to content

Fixes missassignment of n_faces and n_nodes in fesom_to_ugrid - #2810

Open
wyatt-fluidnumerics wants to merge 1 commit into
mainfrom
fix-nod2-elem-mismatch-in-fesom-to-ugrid
Open

Fixes missassignment of n_faces and n_nodes in fesom_to_ugrid#2810
wyatt-fluidnumerics wants to merge 1 commit into
mainfrom
fix-nod2-elem-mismatch-in-fesom-to-ugrid

Conversation

@wyatt-fluidnumerics

@wyatt-fluidnumerics wyatt-fluidnumerics commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Description

Previously, there was a bug in fesom_to_ugrid which mismatched the assignment of n_node and n_face. This PR resolves this issue thereby fixing downstream IndexSearch errors that occur during interpolation.

Checklist

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.
    • Describe how you used it (e.g., by pasting your prompt): This bug was originally found by claude while working on a local copy of the parcels-benchmarks repo. I then verified it and provided the fix myself.

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

Bug in assignment of faces vs nodes in fesom_to_ugrid()

1 participant