Skip to content

Use zstd with compression 15 by default#16

Merged
m-mohr merged 5 commits intomainfrom
use_zstd_by_default
Feb 13, 2026
Merged

Use zstd with compression 15 by default#16
m-mohr merged 5 commits intomainfrom
use_zstd_by_default

Conversation

@ivorbosloper
Copy link
Copy Markdown
Collaborator

closes #12

@ivorbosloper ivorbosloper requested a review from m-mohr October 30, 2025 13:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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_level parameter throughout the conversion pipeline with default of 15 for zstd
  • Added --compression_level CLI 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.

Comment thread vecorel_cli/encoding/geoparquet.py Outdated
Comment thread vecorel_cli/conversion/base.py Outdated
Comment thread vecorel_cli/cli/options.py Outdated
Comment thread vecorel_cli/encoding/geoparquet.py Outdated
Comment thread vecorel_cli/cli/options.py
m-mohr and others added 2 commits February 13, 2026 13:37
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@m-mohr m-mohr merged commit 5ed6ae6 into main Feb 13, 2026
7 checks passed
@m-mohr m-mohr deleted the use_zstd_by_default branch February 13, 2026 12:42
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.

Use zstd with compression 15 by default

3 participants