Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR changes the default compression algorithm for GeoParquet files from Brotli to zstd with a compression level of 15, aligning with the latest recommendations from the GeoParquet community for geospatial data compression. The change affects both the default behavior and adds a new CLI option to allow users to customize the compression level.
Changes:
- Default compression changed from "brotli" to "zstd" for GeoParquet files
- Added
compression_levelparameter throughout the conversion pipeline with default of 15 for zstd - Added
--compression_levelCLI option to the convert command
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| vecorel_cli/encoding/geoparquet.py | Updated write method to use "zstd" as default compression and automatically set compression_level to 15 for zstd when not specified |
| vecorel_cli/cli/options.py | Added GEOPARQUET_COMPRESSION_LEVEL option and changed default compression to "zstd" |
| vecorel_cli/convert.py | Added GEOPARQUET_COMPRESSION_LEVEL to imports and CLI arguments |
| vecorel_cli/conversion/base.py | Added compression_level parameter to convert method and passed it through to GeoParquet.write() |
| CHANGELOG.md | Documented the compression changes in the Unreleased section |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
m-mohr
approved these changes
Feb 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #12