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

MODIS collection errors #371

Closed
JonDHo opened this issue Nov 5, 2024 · 4 comments
Closed

MODIS collection errors #371

JonDHo opened this issue Nov 5, 2024 · 4 comments
Labels

Comments

@JonDHo
Copy link

JonDHo commented Nov 5, 2024

There seem to currently be various issues with STAC records, but I thought I would add this as another report as it doesn't relate to the HLS issues reported previously.

The /items endpoint for some MODIS collections fails completely and searches often fail. For example, the collection endpoint at https://cmr.earthdata.nasa.gov/stac/OB_DAAC/collections/MODISA_L2_OC_R2022.0 works, but https://cmr.earthdata.nasa.gov/stac/OB_DAAC/collections/MODISA_L2_OC_R2022.0/items returns the "Oops! Something has gone wrong." error.

The same goes for other MODIS collections like:

but the SST collections work:

I frequently get the "Oops!" error when searching using PySTAC and PySTAC Client as well (which should be using POST requests).

@JonDHo JonDHo changed the title Missing MODIS OC data MODIS collection errors Nov 5, 2024
@aliciaaleman
Copy link

@JonDHo We have a fix working through our test environment that may resolve some of these issues but we'll confirm before closing this out.

@earthdata-github-robot
Copy link
Collaborator

(Comment from Eudoro Olivares Raniolo):

The error is coming from a graphql typing issue 

Int cannot represent non-integer value

on the cloudCover field. The linked ticket should fix that problem as it propagates into production Wed December 4th

@eudoroolivares2016
Copy link
Contributor

eudoroolivares2016 commented Dec 4, 2024

This is now be fixed in production all those links have valid responses. For those wondering it was an issue with a type in an upstream dependency that cmr-stac has

@JonDHo
Copy link
Author

JonDHo commented Dec 5, 2024

Thanks. I can confirm that this is now working.

In my testing, I note that the number of features returned per page has now moved from the numberReturned value to the context object. It appears that the context extension is being or has been deprecated (radiantearth/stac-api-spec#396). Hopefully if the context object is ultimately removed, it will be possible to add back in the numberReturned and numberMatched values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants