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

/datasets/wble endpoint does not return Great Lakes Water Quality datasets #451

Closed
leothomas opened this issue Nov 11, 2020 · 0 comments · Fixed by NASA-IMPACT/covid-api#81
Assignees
Labels
bug Something isn't working

Comments

@leothomas
Copy link
Collaborator

leothomas commented Nov 11, 2020

Current Behaviour:

The /datasets/wble endpoint returns only global datasets

Desired Behaviour:

The datasets/wble endpoint should return the following datasets:

  • wq-gl-chla
  • wq-gl-spm
  • global datasets

Cause:

In order to return metadata of only datasets that contain data for a particular spotlight, the API searches for the spotlightId value within the tile.url field of each dataset metadata object. Neither wq-gl-chl.json nor wq-gl-spm.json contain the string wble (spotlightId for Great Lakes) within their tile.url fields (

{api_url}/{z}/{x}/{y}@1x?url=s3://covid-eo-data/wq-greatlakes-chl/chl_anomaly_greatlakes_{date}.tif&resampling_method=bilinear&bidx=1&rescale=-100%2C100&color_map=rdbu_r"` and 

and

"{api_url}/{z}/{x}/{y}@1x?url=s3://covid-eo-data/wq-greatlakes-sm/sm_anomaly_greatlakes_{date}.tif&resampling_method=bilinear&bidx=1&rescale=-100%2C100&color_map=rdbu_r"

respectively.

Possible fixes:

  • Rename the spotlight to greatlakes
  • Rename the spotlight to gl and rename the wq-greatlakes-chl and wq-greatlakes-sm folders to wq-gl-chl and wq-gl-spm , respectively
  • Add the string wble somewhere within the filesnames for all wq-greatlakes-chl/* and wq-greatlakes-sm/* filles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant