feat: permissive extent deserialization

[gadomski]

Related Issue(s):

• Closes Relax Collection.from_dict #1558

Description:

Relax collection extent serialization, as there's a surprising number of Collections out there in the real world without extents. Includes a sidecar relaxing of the type on the extent constructors.

PR Checklist:

• Pre-commit hooks pass (run pre-commit run --all-files)
• Tests pass (run pytest)
• This PR maintains or improves overall codebase code coverage.
• Changes are added to the CHANGELOG. See the docs for information about adding to the changelog.

[jsignell]

This looks good! Just one suggestion.

Also this is a larger topic, but I was mildly persuaded at the STAC birds of a feather at CNG that if we are less permissive then it might make users complain to providers and encourage the production of higher quality STAC catalogs.

[gadomski]

I was mildly persuaded at the STAC birds of a feather at CNG that if we are less permissive then it might make users complain to providers and encourage the production of higher quality STAC catalogs.

I get it, but so often the people creating broken STAC are wayyy separated from the feedback system. To me, warning feels like a nice compromise -- if pystac breaks on bad STAC, I think folks are just as likely to just not use pystac-client, make HTTP calls directly to dictionaries, and muddle on.

[jsignell]

pystac breaks on bad STAC, I think folks are just as likely to just not use pystac-client, make HTTP calls directly to dictionaries, and muddle on.

Yeah that makes a ton of sense to me as well. I agree that is the more likely the path that people will see and once we lose them they are probably hard to get back.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.43%. Comparing base (fa00c5d) to head (0d03ae6).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1559      +/-   ##
==========================================
+ Coverage   92.42%   92.43%   +0.01%     
==========================================
  Files          55       55              
  Lines        8368     8371       +3     
  Branches      963      964       +1     
==========================================
+ Hits         7734     7738       +4     
  Misses        449      449              
+ Partials      185      184       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.