search

intake / intake-stac

Intake interface to STAC data catalogs
https://intake-stac.readthedocs.io/en/latest/
BSD 2-Clause "Simplified" License
108 stars 25 forks source link

add "application/xml" to dictionary of drivers #98

Closed fnattino closed 3 years ago

fnattino commented 3 years ago

Fix for #97

TomAugspurger commented 3 years ago

Thanks, can you add a small test for this?

And if you merge main and repush then the CI failures should be fixed.

fnattino commented 3 years ago

Hi @TomAugspurger , sure! I have added a test based on the single-file STAC that was already in the repo, checking whether the media types for all assets are listed in the drivers dictionary. I have also thought it was a good idea to check whether all common media types listed in PySTAC are also listed among the drivers, and realised that image/tiff is actually missing. Should I add it to the dictionary as well (I guess with rasterio as driver)?

TomAugspurger commented 3 years ago

Thanks, that would be great. Either here or as a followup would be fine.

TomAugspurger commented 3 years ago

Thanks @fnattino!