Skip to content

precompute bbox#1293

Draft
DIDONAX wants to merge 1 commit intomotis-project:masterfrom
DIDONAX:precompute-bbox
Draft

precompute bbox#1293
DIDONAX wants to merge 1 commit intomotis-project:masterfrom
DIDONAX:precompute-bbox

Conversation

@DIDONAX
Copy link
Copy Markdown
Contributor

@DIDONAX DIDONAX commented Feb 14, 2026

@felixguendling
Copy link
Copy Markdown
Member

if it's stored in nigiri::timetable it should be computed in init_finish.cc.

The name tt.bbox_ implies that it's a bounding box covering every stop/trip/shape of the timetable which is not true.

@DIDONAX
Copy link
Copy Markdown
Contributor Author

DIDONAX commented Feb 22, 2026

if it's stored in nigiri::timetable it should be computed in init_finish.cc.

in this case, would it be better to rewrite the necessary part from tiles in nigiri (~ 15 lines of code), or move them permanently to the geo library since it's already included in nigiri as dependency?

@felixguendling
Copy link
Copy Markdown
Member

I think they can be moved to geo.

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.

/map/initial: precompute bbox

2 participants