Skip to content

Conversation

@TomAugspurger
Copy link
Collaborator

This updates the file metadata to

  1. Add a version (currently 1.0)
  2. Add stac:geoparquet_version
  3. Deprecate stac:collection
  4. Add a jsonschema file

cc @gadomski. Two questions:

  1. Do we set the version to 1.0 or 1.1? feat: add stac:collections to spec #89 went with 1.1. I can split this in two if you want a 1.0 with stac:geoparquet_version only, cut a 1.0 release, and then follow up with a 1.1 release that has stac:collections? That would probably be best.
  2. Does this match what you have for the rust library?

Supercedes #89

This updates the file metadata to

1. Add a version (currently 1.0)
2. Add `stac:collections`
3. Deprecate `stac:collection`
4. Add a jsonschema file

Supercedes stac-utils#89
@gadomski gadomski self-requested a review May 5, 2025 12:26
Copy link
Member

@gadomski gadomski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can split this in two if you want a 1.0 with stac:geoparquet_version only, cut a 1.0 release, and then follow up with a 1.1 release that has stac:collections? That would probably be best.

Makes sense to me.

Does this match what you have for the rust library?

Not at the moment, I have stac_geoparquet:version: https://github.com/stac-utils/rustac/blob/c8f20c3202e4e8c37e81d7c47644a272719d863b/crates/core/src/geoarrow/mod.rs#L20. But I can update the Rust side.

@TomAugspurger
Copy link
Collaborator Author

OK, I've reverted all the stac:collections (plural) changes for now and will do that separately. This should just have the changes to

  1. Add jsonschema for the parquet file metadata
  2. Add stac_geoparquet:version to the spec

@TomAugspurger
Copy link
Collaborator Author

Planning to merge this tonight. Follow up tasks

  1. Cut a release (version TBD. More evidence for tying the spec to the (python) code repo being problematic.)
  2. Make a PR adding collections and deprecating collection.

@TomAugspurger TomAugspurger merged commit 6c1d673 into stac-utils:main May 14, 2025
4 checks passed
@TomAugspurger TomAugspurger deleted the 94-metadata-versioned branch May 14, 2025 11:26
This was referenced May 14, 2025
gadomski added a commit to stac-utils/rustac that referenced this pull request May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants