Skip to content

Updates for PySTAC 0.3.1 #40

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.

Sign up for GitHub

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

Jump to bottom
Merged
lossyrob merged 11 commits into from
Nov 5, 2019
Merged

Updates for PySTAC 0.3.1 #40

lossyrob merged 11 commits into from
Nov 5, 2019

Conversation

@lossyrob
Copy link
Member

@lossyrob lossyrob commented Nov 5, 2019

Changes in this PR:

  • Removed some of the validation logic around LabelItem, which can feel restrictive in the case where GeoJSON labels have an implicit class without a Feature property that states the class (e.g. SpaceNet 5 doesn't have a 'roads' class in a label property; it's known that the GeoJSON contains roads). I think some tweaking of the requirements of the label extension spec has to be done.
  • Made some backwards compatibility changes for 0.8.0 -> 0.8.1 due to Pluralized forms of fields radiantearth/stac-spec#597 to reduce user pain.
  • Move STAC_VERSION to 0.8.1, all links and references now point to that tag.
  • Check stac_extensions when determining what class to deserialize. This was necessary for an instance where the Item properties didn't have any eo: properties, but the collection held all eo: properties - these should still deserialize to EOItem.
  • Fixed an issue where cleared items and children were still being held in the ResolvedObjectCache of a Catalog.

lossyrob added 11 commits November 4, 2019 14:13
@lossyrob
Remove validation for LabelItem around Classes requirement.
1c5e37b 
There's still some work to determine just when classes and properties
are required: specifically, in the case where labels are Features with
implied classes (e.g. Spacenet 5 roads).
@lossyrob
Add missing pydocs.
e83d503
@lossyrob
Update STAC version to 0.8.1
a37c63a
@lossyrob
Move all links and stac versions to 0.8.1
74ec1d9
@lossyrob
Allow label to be read with 0.8.0 non-pluralized properties.
69ee5c6 
See https://github.com/radiantearth/stac-spec/pull/597/files
@lossyrob
Check stac_extension for extended items.
bc9d296 
Otherwise, properties inherited by the collection can cause extended
objects to be parsed as standard objects.
@lossyrob
Fix issue where cleared items are still in root object cache.
48d1836
@lossyrob
Add methods for removing single items and children.
7455a2e
@lossyrob
Style fixes.
60b1a13
@lossyrob
Update CHANGELOG.
a12d5f3
@lossyrob
Version 0.3.1
ca6c996
@lossyrob lossyrob merged commit c3ea2b4 into develop Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lossyrob