Skip to content
This repository has been archived by the owner on Nov 7, 2019. It is now read-only.

[Not Ready] use Tile Reduce v3, improve false-positive matching, speed #7

Merged
merged 8 commits into from
Nov 23, 2015

Conversation

tcql
Copy link
Contributor

@tcql tcql commented Nov 9, 2015

@mourner
Copy link
Member

mourner commented Nov 9, 2015

  • first gather an array of bbox+id objects, then do roadIndex.load(items) — bulk loading is faster than insert one by one
  • try inserting segments separately (they don't have to be combined for intersection tests) — smaller bboxes means more efficient searches
  • avoid concat in places where you could just do push (doing less copy)

@tcql
Copy link
Contributor Author

tcql commented Nov 23, 2015

TileReduce v3 is 👍, just need to incorporate @mourner's suggestions

@mourner
Copy link
Member

mourner commented Nov 23, 2015

You can merge now and implement suggestions later.

tcql added a commit that referenced this pull request Nov 23, 2015
Upgraded to TileReduce v3. Improved false-positive matching & speed
@tcql tcql merged commit 83f2a7f into master Nov 23, 2015
@tcql tcql deleted the tr-v3 branch November 23, 2015 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants