podaac / l2ss-py

Level 2 subsetter with Harmony integration
https://podaac.github.io/l2ss-py/
Apache License 2.0
11 stars 12 forks source link

issue-78: Harmony adapter passes coordinate variables to l2ss-py #90

Closed skorper closed 2 years ago

skorper commented 2 years ago

Github Issue: #78

Description

Harmony adapter passes coordinate variables to l2ss-py

Overview of work done

Overview of verification done

Overview of integration done

http://localhost:3000/C1234724470-POCLOUD/ogc-api-coverages/1.0.0/collections/all/coverage/rangeset?subset=lat(0%3A90)&maxResults=2

You can see in the log message that I added that the lat/lon/time vars are passed in:

2022-05-23T19:01:10.346Z [debug]: INFO:root:Calling l2ss-py subset with params {'bbox': array([[-180,  180],
       [   0,   90]]), 'lat_var_names': ['lat'], 'lon_var_names': ['lon'], 'time_var_names': ['time'], 'output_file': '/home/dockeruser/data/81066b2831bab8ab47621d552e97e029673fe6d1cfb59ff2d5fa2ca926c1ec6b.nc', 'file_to_subset': '/tmp/tmp7pq3445o/81066b2831bab8ab47621d552e97e029673fe6d1cfb59ff2d5fa2ca926c1ec6b.nc', 'origin_source': 'https://archive.podaac.uat.earthdata.nasa.gov/podaac-uat-cumulus-protected/MODIS_A-JPL-L2P-v2019.0/20020704000506-JPL-L2P_GHRSST-SSTskin-MODIS_A-N-v02.0-fv01.0.nc'}
{"message": "Calling l2ss-py subset with params {'bbox': array([[-180,  180],\n       [   0,   90]]), 'lat_var_names': ['lat'], 'lon_var_names': ['lon'], 'time_var_names': ['time'], 'output_file': '/home/dockeruser/data/81066b2831bab8ab47621d552e97e029673fe6d1cfb59ff2d5fa2ca926c1ec6b.nc', 'file_to_subset': '/tmp/tmp7pq3445o/81066b2831bab8ab47621d552e97e029673fe6d1cfb59ff2d5fa2ca926c1ec6b.nc', 'origin_source': 'https://archive.podaac.uat.earthdata.nasa.gov/podaac-uat-cumulus-protected/MODIS_A-JPL-L2P-v2019.0/20020704000506-JPL-L2P_GHRSST-SSTskin-MODIS_A-N-v02.0-fv01.0.nc'}", "user": "skperez", "requestId": "d2e4f851-22c7-4e9d-b579-dd107e216748", "timestamp": "2022-05-23T19:01:10.344380Z", "level": "INFO", "application": "/home/dockeruser/.local/bin/l2ss_harmony"}

PR checklist:

See Pull Request Review Checklist for pointers on reviewing this pull request