Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Working on ensuring that all the points in zoom N are always also in zoom N+1 #200

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

Working on ensuring that all the points in zoom N are always also in zoom N+1 #200

wants to merge 20 commits into from

Conversation

e-n-f
Copy link
Collaborator

@e-n-f e-n-f commented Feb 15, 2024

No description provided.

@e-n-f
Postpone tagging features as being the first of a multiplier cluster
10257e1
@e-n-f
Upgrade some dynamically dropped features to multiplier features
6a6cd55
@e-n-f
Still don't let it put more features in a cluster than is allowed
fbafe3d
@e-n-f
Update tests
40793a4
@e-n-f
Make the current multiplier cluster size per-layer
65a1d0a
@e-n-f
Make sure the first non-empty-geometry in the layer is marked as primary
0e3ac8e
@e-n-f
Improve comments
2982b1b
@e-n-f
Update changelog and version
5df9a5a
@e-n-f
Merge remote-tracking branch 'origin/main' into keep-more-filter-feat…
90f87c1 
…ures
@e-n-f
Script to make sure that points are nested correctly between zooms
44879aa
@e-n-f
Don't always keep the first feature in a tile, after all
fcecdf9
@e-n-f
Make sure at least one feature appears in every zoom level
159abfe
@e-n-f
All the point tests change
3839c8b
@e-n-f
Don't crash when there are no tiles in a zoom level
a60954a
@e-n-f
Merge branch 'keep-more-filter-features' into zoom-nesting
0bb1d89
@e-n-f e-n-f changed the base branch from main to keep-more-filter-features February 15, 2024 16:31
Base automatically changed from keep-more-filter-features to main February 15, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@migurski migurski Awaiting requested review from migurski migurski will be requested when the pull request is marked ready for review migurski is a code owner

@arredond arredond Awaiting requested review from arredond arredond will be requested when the pull request is marked ready for review arredond is a code owner

@dnomadb dnomadb Awaiting requested review from dnomadb dnomadb will be requested when the pull request is marked ready for review dnomadb is a code owner

@ChrisLoer ChrisLoer Awaiting requested review from ChrisLoer ChrisLoer will be requested when the pull request is marked ready for review ChrisLoer is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@e-n-f