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

allow to use a different masking value than missing #23

Merged
merged 2 commits into from
Feb 16, 2024

Conversation

Alexander-Barth
Copy link
Member

@Alexander-Barth Alexander-Barth commented Feb 6, 2024

Currently missing values (and fill values) are loaded as missing in Julia. For some use cases, it is preferable to use different special values like NaNs (e.g. for GPUs, or interfacing with other C libraries).

This PR allows the user to e.g. use NaN either per variable or per datasets.

What do you think?

@tcarion
Copy link
Member

tcarion commented Feb 7, 2024

I think that's great, thank you for the PR !

@tcarion
Copy link
Member

tcarion commented Feb 16, 2024

Sorry @Alexander-Barth, I think I forgot to merge this ! Could you reopen it ?

@Alexander-Barth
Copy link
Member Author

Alexander-Barth commented Feb 16, 2024

Oh, sorry. I just synchronized my branch with your main. I forgot that this was not yet merged. PR got closed automatically by the syncing the branches. I will re-open or make a new PR.

@Alexander-Barth
Copy link
Member Author

OK, just clicking on reopen was enough to revive the PR :-)

@tcarion tcarion merged commit 98088ad into JuliaGeo:main Feb 16, 2024
6 of 10 checks passed
@Alexander-Barth
Copy link
Member Author

Thanks and have a nice week-end!

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

Successfully merging this pull request may close these issues.

2 participants