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

Individual process testing updates #7

Merged
merged 12 commits into from
Dec 22, 2023
Merged

Individual process testing updates #7

merged 12 commits into from
Dec 22, 2023

Conversation

m-mohr
Copy link
Member

@m-mohr m-mohr commented Dec 13, 2023 •
edited
Loading

Updates to the test suite (individual process testing)

@m-mohr m-mohr force-pushed the indiv-processes branch 4 times, most recently from 278bb89 to 53f228a Compare December 14, 2023 16:12
@m-mohr m-mohr requested a review from soxofaan December 14, 2023 16:17
@m-mohr m-mohr marked this pull request as ready for review December 14, 2023 23:43
@m-mohr
Copy link
Member Author

m-mohr commented Dec 14, 2023 •
edited
Loading

Next version which could be merged pending your review @soxofaan

Open ToDos (for after this PR):

  • Add tests for filter_bbox, filter_spatial, mask, resample_spatial, merge_cubes, resample_cube_spatial
  • Update openeo-processes descriptions and tests with regards to null vs NaN openeo-processes#480

So this is mostly in openeo-processes, I don't anticipate a lot more changes in the test suite itself right now.

@m-mohr m-mohr merged commit d486ae7 into main Dec 22, 2023
@m-mohr m-mohr deleted the indiv-processes branch December 22, 2023 20:09
@m-mohr
Copy link
Member Author

m-mohr commented Dec 22, 2023

Merging as (officially) this expected to be available before end of the year.

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

@soxofaan soxofaan Awaiting requested review from soxofaan

Assignees
No one assigned
Labels
WP5: individual processes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@m-mohr @soxofaan