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

DAS-2293: Add SPL2SMA regression test for smap-l2-gridder #134

Merged
merged 3 commits into from
Feb 6, 2025

Conversation

flamingbear
Copy link
Member

@flamingbear flamingbear commented Feb 5, 2025

Description

A short description of the changes in this PR...

Jira Issue ID

DAS-2293

Local Test Steps

Ensure you've beefed up your docker environment according to the instructions in nasa/harmony-SMAP-L2-gridding-service#17, deploy that service image to your Harmony-In-A-Box.

Pull this branch and run the regression tests.

PR Acceptance Checklist

  • Acceptance criteria met
  • Tests added/updated (if needed) and passing
  • Documentation updated (if needed)
  • CHANGELOG updated with the changes for this PR

@flamingbear flamingbear marked this pull request as ready for review February 5, 2025 22:16
@flamingbear flamingbear requested review from owenlittlejohns and a team February 5, 2025 22:17
Copy link
Member

@owenlittlejohns owenlittlejohns left a comment

Choose a reason for hiding this comment

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

Nice changes - the setup of the notebook made it really easy to drop those in. Nice.

12_SMAP_L2_SM_A_01058_D_20150413T220200_R13080_001_regridded.nc
Downloaded: 12_SMAP_L2_SM_A_01058_D_20150413T220200_R13080_001_regridded.nc
Saved output to: /var/folders/x_/f490b72n0qn1p67zpmr71bpr0000gp/T/tmpft8fn7s3/SPL2SMA.nc
Success: SPL2SMA Test.
Success: Entire Test Suite.

@flamingbear flamingbear merged commit 75d4fbe into main Feb 6, 2025
1 check passed
@flamingbear flamingbear deleted the mhs/DAS-2293/smap-l2-gridder-add-spl2sma-tests branch February 6, 2025 23:24
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