Skip to content

Merge clarifications and editorial improvements - #8

Open
m-mohr wants to merge 1 commit into
mainfrom
merge-clarifications
Open

m-mohr wants to merge 1 commit into
mainfrom
merge-clarifications

Conversation

@m-mohr

@m-mohr m-mohr commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

Related Issue(s): None. Came up while reviewing the merge implementation in vecorel/cli (vecorel/cli#55 and a follow-up PR).

Proposed Changes:

  1. Add a "Merging collections" section to the core specification. It covers the collection property at the feature level, combining the schemas of a collection that occurs in multiple datasets, keeping id unique per collection, moving collection-level properties back to the feature level when their values no longer apply to all features, and removing collection-only properties whose values differ.
  2. Clarify that collection-level data applies to all features, even across collections, with schemas as the only exception.
  3. Collection-level data needs at least two features (it said "more than two").
  4. Fix the GeoParquet bit width of uint32, which is 32 (it said 64).
  5. Editorial fixes:
    • schemas is described as an object per collection, not a list.
    • A Feature is described as a "single geometry" instead of a "single field geometry".
    • Removed a duplicate sentence about the required core schema.
    • Fixed an invalid JSON example (trailing comma).
    • The GeoJSON encoding now refers to GeoJSON data types instead of Parquet data types.
    • Fixed typos ("provieded", "sturcture", "can#t", "defines to encode").

PR Checklist:

  • This PR has no breaking changes.
  • I have added my changes to the CHANGELOG
    or a CHANGELOG entry is not required.

This comment was marked as low quality.

This branch has not been deployed

No deployments
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.

2 participants