Skip to content

[5.x] update zarr test data#776

Merged
lesserwhirls merged 1 commit intoUnidata:maint-5.xfrom
haileyajohnson:test-data
Aug 4, 2021
Merged

[5.x] update zarr test data#776
lesserwhirls merged 1 commit intoUnidata:maint-5.xfrom
haileyajohnson:test-data

Conversation

@haileyajohnson
Copy link
Copy Markdown
Contributor

@haileyajohnson haileyajohnson commented Aug 4, 2021

Description of Changes

  • reduce number of files (chunks) in test data set
  • add extra test cases (nested array, uninitialized array, partially initialized array, etc.)
  • remove compressor from current test data (compression to be tested separately)

PR Checklist

  • Indicate the version associated with this PR in the Title
    (e.g. "[5.x]: This is my PR title")
  • Link to any issues that the PR addresses
  • Add labels, especially if the PR should be ported to other versions
    (these labels start with "port: ")
  • Open as a draft PR
    until ready for review
  • Make sure GitHub tests pass
  • Mark PR as "Ready for Review"

This change is Reviewable

@haileyajohnson haileyajohnson marked this pull request as ready for review August 4, 2021 21:44
Copy link
Copy Markdown
Member

@lesserwhirls lesserwhirls left a comment

Choose a reason for hiding this comment

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

Looks good!

Reviewed 1053 of 1053 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @dopplershift)

@lesserwhirls lesserwhirls merged commit 2314e38 into Unidata:maint-5.x Aug 4, 2021
@haileyajohnson haileyajohnson deleted the test-data branch November 16, 2021 22:45
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