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

Change NUOPC default to unmappedAction=error #319

Open
danrosen25 opened this issue Nov 6, 2024 · 1 comment
Open

Change NUOPC default to unmappedAction=error #319

danrosen25 opened this issue Nov 6, 2024 · 1 comment
Assignees
Labels

Comments

@danrosen25
Copy link
Member

The initial release of NUOPC set the default connector unmappedAction to ignore. In the next major release (9.0.0) we should change the default connector unmappedAction to error. Users should be aware of situations where regridding is incomplete. The simple solution is for users to set an extrapolation method that will map any previously unmapped cells. The more advanced solution is to intentionally ignore unmapped cells and handle them in user code (this can be done with fill values and zeroRegion select).

@danrosen25 danrosen25 self-assigned this Nov 6, 2024
@anntsay
Copy link

anntsay commented Nov 13, 2024

If change the default -> it would not be backward compatible.. and it is an interface change. it would have to be a major release in 9.0.0.

caution: this may break NUOPC app prototypes.

will plan to give community heads up.

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

No branches or pull requests

2 participants