Skip to content

[WIP] PDOK-18835/vervanging-trex-v2-vertegeling - #42

Draft
DvBree wants to merge 137 commits into
masterfrom
PDOK-18835/vervanging-trex-v2-vertegeling
Draft

DvBree wants to merge 137 commits into
masterfrom
PDOK-18835/vervanging-trex-v2-vertegeling

Conversation

@DvBree

@DvBree DvBree commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Description

What have you changed and why?

Part of https://kadaster.atlassian.net/browse/PDOK-19167. This is a first step with the following functionality:

  • add flag 'enc' indicating tiles should be encoded.
  • basic detection which geometries are hit by which tiles (bbox without buffer)
  • encode geometries according to mapbox specs
  • store encoded geometries in new table "TABLENAME_encoded'.

Type of change

(Remove irrelevant options)

  • New feature

Checklist:

  • I've double-checked the code in this PR myself
  • I've left the code better than before (boy scout rule)
  • The code is readable, comments are added that explain hard or non-obvious parts.
  • I've expanded/improved the (unit) tests, when applicable
  • I've run (unit) tests that prove my solution works
  • There's no sensitive information like credentials in my PR

Comment thread pointindex/pointindex.go
}

func (q *Quadrant) Coords() (uint, uint) {
return morton.FromZ(q.z)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

*morten

Comment thread processing/processing.go Outdated
DvBree and others added 30 commits September 10, 2026 10:18
…-and-lines

Pdok 18835/vervanging trex v2 points and lines
…ithout-geoms

Pdok 18835/vervanging trex v2 exit without geoms
…ng-coords

fix: round floats with floor/ceil instead of truncation
…-blobstore

Pdok 18835/vervanging trex v2 target blobstore
This is relevant for testing: many tests do not include all layers in the gpkg
…g-layers

fix: do not fail when config layers are missing in gpkg
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.

3 participants