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.

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

Resample dask arrays blockwise using pyresample #128

Merged
merged 17 commits into from
Jun 17, 2024
Merged

Resample dask arrays blockwise using pyresample #128

merged 17 commits into from
Jun 17, 2024

Conversation

maxrjones
Copy link
Contributor

@maxrjones maxrjones commented May 25, 2024 •
edited
Loading

New attempt at implementing #10 using pyresample's resample_blocks.

To-do:

  • Test values of pyramid data in addition to coords
  • Infer x/y dimensions from CF conventions
  • Add test for non-CF compliant data
  • Test 2D dataset
  • Add documentation. Note: Include a warning about dims order (time,y,x)

I think we could add support for 4D data as a separate PR.

@maxrjones maxrjones marked this pull request as draft May 25, 2024 00:34
@norlandrhagen norlandrhagen mentioned this pull request Jun 4, 2024
Closed
@maxrjones maxrjones mentioned this pull request Jun 4, 2024
Closed
@maxrjones maxrjones mentioned this pull request Jun 13, 2024
Open
@maxrjones maxrjones changed the title WIP: Resample dask arrays blockwise using pyresample Resample dask arrays blockwise using pyresample Jun 17, 2024
@maxrjones maxrjones marked this pull request as ready for review June 17, 2024 16:27
norlandrhagen
norlandrhagen approved these changes Jun 17, 2024
View reviewed changes
Copy link
Contributor

@norlandrhagen norlandrhagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great Max!

@maxrjones maxrjones merged commit 5bfeaec into main Jun 17, 2024
8 checks passed
@maxrjones maxrjones deleted the resample-blocks branch June 17, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@norlandrhagen norlandrhagen norlandrhagen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maxrjones @norlandrhagen