Skip to content

Reference files to test ObsSpace::reduce and ObsSpace::append for OSDF#231

Open
lewisn-met wants to merge 7 commits intodevelopfrom
feature/osdf-reduce
Open

Reference files to test ObsSpace::reduce and ObsSpace::append for OSDF#231
lewisn-met wants to merge 7 commits intodevelopfrom
feature/osdf-reduce

Conversation

@lewisn-met
Copy link
Contributor

@lewisn-met lewisn-met commented Jan 19, 2026

Description

Adds reference files to test the reduce functionality of the ObsSpace, when the ObsSpace is OSDF based.

Issue(s) addressed

Resolves JCSDA-internal/ioda#1605

Dependencies

  • JCSDA-internal/ioda#1616 (ioda PR must be merged before this PR)
  • JCSDA-internal/ioda#1623 (ioda PR must be merged before this PR)
  • JCSDA-internal/ioda#1617 (coordinated merge)
  • JCSDA-internal/ioda#1550 (issue needs to be resolved before merge)

Impact

N/A (OSDF functionality not currently implemented)

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

@lewisn-met lewisn-met self-assigned this Jan 19, 2026
@lewisn-met lewisn-met added coordinate merge Ready for merge but needs to be coordinated with other repos UKMO UK Met Office INTERFACE JEDI Models Interface labels Jan 19, 2026
@lewisn-met lewisn-met changed the title Reference files to test ObsSpace::reduce for OSDF Reference files to test ObsSpace::reduce and ObsSpace::append for OSDF Jan 29, 2026
@lewisn-met
Copy link
Contributor Author

Added reference files for testing ObsSpace::append to this PR, since they have the same requirements.

@lewisn-met
Copy link
Contributor Author

As per the discussion in JCSDA-internal/ioda/pull/1684, this data update may not be necessary (as it duplicates reference files already used for ObsGroup tests). Will test once the OSDF writer is in place and close the PR if this is the case.

@srherbener
Copy link
Collaborator

As per the discussion in JCSDA-internal/ioda/pull/1684, this data update may not be necessary (as it duplicates reference files already used for ObsGroup tests). Will test once the OSDF writer is in place and close the PR if this is the case.

Sounds great! Thanks

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

Labels

coordinate merge Ready for merge but needs to be coordinated with other repos INTERFACE JEDI Models Interface UKMO UK Met Office

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants