Failing test: test_load_dataset_from_stac_plugin

[ots22]

$ python -m pytest -k test_load_dataset_from_stac_plugin
FAILED tests/test_reader.py::test_load_dataset_from_stac_plugin - pystac_client.warnings.DoesNotConformTo: Server does not conform to ITEM_SEARCH, There is not fallback option available for search.

Failure suggests something upstream has changed/broken

[IFenton]

@ots22 Would you like me to look in to this?

[ots22]

Hi @IFenton - some progress so far:

Directly using the pystac_client library

>>> from pystac_client import Client
>>> catalog = Client.open("https://earth-search.aws.element84.com/v0")
>>> catalog.search()

fails with DoseNotConformTo exception.

The 'v1' API does not have this issue (see https://www.element84.com/earth-search/examples/):

>>> catalog = Client.open("https://earth-search.aws.element84.com/v1")
>>> query = catalog.search(collections=["sentinel-s2-l2a-cogs"], datetime=["2021-09-16"], bbox=(113.0,-26.0,114.0,-25.0), limit=10)
>>> list(query.items())

The resulting query is empty, however - I haven't looked at why that could be

[ots22]

Suggestions:

• Raise the above in https://github.com/alan-turing-institute/scivision_sentinel2_stac (changing the URL to v1 might be the fix)
• Remove the failing test (but make sure this is included in the catalog - this kind of failure should be picked up by Automated Workflow Checks for Data and Models #566)

[ots22]

...make sure this is included in the catalog...

It is in the data catalogue already, as sentinel2_stac

[IFenton]

The 'v1' API does not have this issue (see https://www.element84.com/earth-search/examples/):

catalog = Client.open("https://earth-search.aws.element84.com/v1")
query = catalog.search(collections=["sentinel-s2-l2a-cogs"], datetime=["2021-09-16"], bbox=(113.0,-26.0,114.0,-25.0), limit=10)
list(query.items())

The resulting query is empty, however - I haven't looked at why that could be

It seems to be because v1 doesn't have the relevant collection ('sentinel-s2-l2a-cogs')

[ots22]

Ah, that's why then!

The above PR removes the test since it isn't failing in a way that gives useful information - do you think this is the right approach @IFenton? (Please could you 'review'?)

The issue really belongs to https://github.com/alan-turing-institute/scivision_sentinel2_stac