Skip to content

Added new meta data to a test netcdf4 file. - #582

Open
samueljmcameron wants to merge 2 commits into
developfrom
feature/filter-profile_threshold_rejection
Open

Added new meta data to a test netcdf4 file.#582
samueljmcameron wants to merge 2 commits into
developfrom
feature/filter-profile_threshold_rejection

Conversation

@samueljmcameron

@samueljmcameron samueljmcameron commented Jul 16, 2026

Copy link
Copy Markdown

Description

This PR adds a new variable, MetaData/ordering, to testinput_tier_1/profile_filter_testdata.nc4.

Issue(s) addressed

Required for JCSDA-internal/ufo#4190

Dependencies

n/a

Impact

There should be no impact as a variable is just added to the file.

Manual Testing Instructions (optional)

If you would like your reviewers to manually build and test the change, please include
instructions on how the change should be built and tested. Also include a short
justification on why manual testing is necessary for this change.

Checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have run the unit tests before creating the PR

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Git LFS-tracked NetCDF4 test input testinput_tier_1/profile_filter_testdata.nc4 to include a new variable (MetaData/ordering) needed for downstream testing referenced by JCSDA-internal/ufo#4190.

Changes:

  • Updated the Git LFS object pointer (oid/size) for profile_filter_testdata.nc4, reflecting new embedded metadata in the binary file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 1 to +3
version https://git-lfs.github.com/spec/v1
oid sha256:d3b0dc2955969d79e8e5e685fdd0d25a7ff9382e654f689d5834ec086dd9978c
size 30522
oid sha256:c8930b0f9f4e21759f6cea5f9e8e5978739e96c6d903ef66f2e4e2ace02cd5c6
size 37130

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like this idea!

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.

4 participants