Skip to content

Giovanni averaging adapter#712

Merged
chris-durbin merged 13 commits intonasa:mainfrom
dieumynguyen:main
Mar 12, 2025
Merged

Giovanni averaging adapter#712
chris-durbin merged 13 commits intonasa:mainfrom
dieumynguyen:main

Conversation

@dieumynguyen
Copy link
Copy Markdown
Contributor

Jira Issue ID

CLDGIANNI-3970

Description

We have prepared and locally tested the Giovanni averaging adapter. Since Harmony has added support for validate_variables: false, we are using our internal data field ID for the variable/UMM-Var ID. We have created the association between the UMM-S record for this service and a collection we test on: C1215802935-GES_DISC. Harmony's AWS non-prod accounts should be able to access Giovanni resources (S3, catalog endpoint) as we have tested last sprint.

Local Test Steps

  • Docker image: We will shortly update this as we push the image to an ECR repo you can pull from.
  • Test URLS:
    • Averaging along time to create a time-averaged map (geotiff file ouput):
      http://localhost:3000/C1215802935-GES_DISC/ogc-api-coverages
      /1.0.0/collections/M2T1NXAER_5_12_4_BCCMASS/coverage/rangeset?format=image%2Ftiff&subset=lat(-50%3A
      -48)&subset=lon(-90%3A
      -87)&subset=time(%222009-01-01T00%3A00%3A00%22%3A%222009-01-05T10%3A31%3A12%22)&average=time

    • Averaging along spatial dimensions to create area-averaged timeseries (csv file ouput):
      http://localhost:3000/C1215802935-GES_DISC/ogc-api-coverages
      /1.0.0/collections/M2T1NXAER_5_12_4_BCCMASS/coverage/rangeset?format=text%2Fcsv&subset=lat(-50%3A
      -48)&subset=lon(-90%3A
      -87)&subset=time(%222009-01-01T00%3A00%3A00%22%3A%222009-01-05T10%3A31%3A12%22)&average=area

PR Acceptance Checklist

  • Acceptance criteria met
  • Tests added/updated (if needed) and passing
  • Documentation updated (if needed)
  • Harmony in a Box tested (if changes made to microservices or new dependencies added)

@chris-durbin chris-durbin merged commit 9c8fd4f into nasa:main Mar 12, 2025
3 of 4 checks passed
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.

3 participants