Skip to content

Update fiboa CLI, update fiboa to v0.3 (breaking)#241

Merged
m-mohr merged 1 commit intomainfrom
update-fiboa
Feb 16, 2026
Merged

Update fiboa CLI, update fiboa to v0.3 (breaking)#241
m-mohr merged 1 commit intomainfrom
update-fiboa

Conversation

@m-mohr
Copy link
Copy Markdown
Collaborator

@m-mohr m-mohr commented Feb 16, 2026

Fixes the pkg_resources issue which prevents ftw-tools to be installed by updating to the most recent fiboa CLI version.

This also updates the fiboa version to 0.3, which means we need to update the app slightly (see fieldsoftheworld/ftw-inference-app#231 )

Waiting for fiboa/cli#202

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 updates ftw-tools to use the latest fiboa CLI (to address the pkg_resources install issue) and adapts the codebase to fiboa v0.3-style field/property naming and GeoParquet writing.

Changes:

  • Bump fiboa-cli dependency to >=0.21,<0.22 and refresh the lockfile accordingly.
  • Update polygon output properties to namespaced fields (e.g., metrics:*, determination:*) and route Parquet writing through the newer fiboa_cli/vecorel_cli pathway.
  • Adjust tests to match the renamed metrics:* properties.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
uv.lock Updates resolved dependency set for the new fiboa-cli version (and transitive deps).
pyproject.toml Bumps fiboa-cli requirement to >=0.21,<0.22.
ftw_tools/postprocess/polygonize.py Renames output properties to metrics:*/determination:* and switches Parquet writing to convert_to_fiboa.
ftw_tools/inference/utils.py Replaces legacy parquet writer usage with Registry + vecorel_cli GeoParquet writing; updates property names.
tests/test_merge_adjacent.py Updates assertions for the new metrics:* properties.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ftw_tools/inference/utils.py
Comment thread ftw_tools/inference/utils.py
Comment thread ftw_tools/postprocess/polygonize.py
Comment thread ftw_tools/postprocess/polygonize.py
Comment thread ftw_tools/postprocess/polygonize.py
@m-mohr m-mohr merged commit 0a58edd into main Feb 16, 2026
7 checks passed
@m-mohr m-mohr deleted the update-fiboa branch February 16, 2026 14:04
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.

Remove flatdict dependency Update fiboa CLI dependency to > 0.20.0 (and thus to fiboa spec 0.3.0)

3 participants