-
Notifications
You must be signed in to change notification settings - Fork 20
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
Use has_extension
rather than deprecated validate_has_extension
#134
Conversation
Thanks for the PR, do you know when |
odc/stac/_mdtools.py
Outdated
ext_name.startswith("https://stac-extensions.github.io/raster/") | ||
for ext_name in item.stac_extensions | ||
) | ||
return RasterExtension.has_extension(item) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are failing because of this change, I realize that newer version of pystac
are more lenient as far as versions of extensions are concerned, but we might need to support older versions for a bit, so can we have that "dodgy check" back?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On second thought, the "dodgy check" is kind of separate. I can add it back in.
Yeah it seems like Would it make sense to just bump up the floor for min pystac supported by this library? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #134 +/- ##
===========================================
- Coverage 87.44% 87.41% -0.03%
===========================================
Files 19 19
Lines 1864 1860 -4
===========================================
- Hits 1630 1626 -4
Misses 234 234 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
With pystac 1.9 there
validate_has_extension
is deprecated, so there was a warning cropping up. This seemed like a simpler solution anyways.