Skip to content

Conversation

@james-willis
Copy link
Contributor

…MTiles

@gitnotebooks
Copy link

gitnotebooks bot commented Dec 17, 2025

Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/wherobots/wherobots-examples/pull/105

Copy link

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 addresses a performance issue with PMTiles generation in the FTW (Field/Territory/Watershed) model notebook by subdividing extremely large boundary geometries that cause problems during tile creation.

Key changes:

  • Adds geometry subdivision logic to handle large boundary polygons before PMTiles generation
  • Separates field and boundary features into distinct layers for tile generation

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

james-willis and others added 2 commits December 17, 2025 11:54
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Copy link
Member

@rbavery rbavery left a comment

Choose a reason for hiding this comment

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

looking forward to subdivide being an automated feature of pmtiles that users don't need to think about

@james-willis
Copy link
Contributor Author

james-willis commented Dec 17, 2025

im not closing the other bug you sent; need to rc why we need to subdivide in the first place.

Its kind of an opinionated thing to do so Im hesitant to automate it into the PMTiles

@james-willis james-willis merged commit 81aa29c into main Dec 17, 2025
3 checks passed
@james-willis james-willis deleted the ftw-tiles-fix branch December 17, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants