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

Create integration test for multiple scan time reader(s) #804

Open
5 tasks
evrose54 opened this issue Oct 23, 2024 · 0 comments
Open
5 tasks

Create integration test for multiple scan time reader(s) #804

evrose54 opened this issue Oct 23, 2024 · 0 comments
Labels
testing Test datasets, scripts, and processes
Milestone

Comments

@evrose54
Copy link
Contributor

Requested Update

Description

PR #427 introduced changes which made some readers able to handle multiple scan times, as opposed to a single scan time. This is needed for certain applications at CIRA (such as OCTANE), but introduces different challenges. To make sure that both single scan time or multi-scan time reads work for these updated readers, we should add a suite of integration tests which address these new changes. Specifically, we should put together a few test datasets which have multiple scan times, add an algorithm which can handle this new 'time' dimension, and produce some output to test against.

We should keep these datasets as small as possible, so it's probably best to use M01 for GOES16 ABI, LA for GEOKOMPSAT AMI,
JP01 for Himawari9 AHI, and other small sectors for Meteosat[X]-SEVIRI.

Background and Motivation

This comes from a conversation that @jsolbrig and I had when testing #427.

Code to demonstrate issue

These readers are able to handle multiple scan time:
image

Checklist for Completion

  • Upload applicable test datasets to CIRA's NextCloud instance (see readers above for what data to upload)
  • Update download test scripts to include these new datasets
  • Create new products for multiple scan times
  • Create a new algorithm which can handle 2 times or more
  • Add new integration tests which test this functionality
@evrose54 evrose54 added the testing Test datasets, scripts, and processes label Oct 23, 2024
@evrose54 evrose54 added this to the 2.0 milestone Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Test datasets, scripts, and processes
Projects
None yet
Development

No branches or pull requests

1 participant