Skip to content

Comments

Search meta tiles in full detail#75

Merged
elliottslaughter merged 4 commits intomasterfrom
full-search
May 10, 2025
Merged

Search meta tiles in full detail#75
elliottslaughter merged 4 commits intomasterfrom
full-search

Conversation

@elliottslaughter
Copy link
Contributor

@elliottslaughter elliottslaughter commented May 9, 2025

Fixes #42 and #45.

  • Slot now tracks separate sets of full and non-full meta tiles.
  • Display functions always refer to non-full meta tile. Tooltips etc. correspond to what is drawn on the screen (i.e., non-full).
  • Search functions always refer to full meta tiles. (I.e., always search in full detail.)
  • request_tiles now takes a full parameter. For static profiles, this always returns the last (i.e., deepest) zoom level.
  • DeferredDataSource now tracks full through the return path so callers can determine what call a given tile corresponds to.

Search will be broken but at least the profiles will load and tooltips
will function.
Static profiles are generated with full at the last level and cannot
respond to a request without, so if we read the value out of the saved
data, we'll get the wrong answer.
@lightsighter
Copy link

Works for me.

@elliottslaughter elliottslaughter merged commit 5f6380d into master May 10, 2025
14 checks passed
@elliottslaughter elliottslaughter deleted the full-search branch May 10, 2025 06:07
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.

Search inside merged tasks

2 participants