nrenner / brouter-web

Web client for BRouter, a routing engine based on OpenStreetMap
https://brouter.de/brouter-web/
MIT License
363 stars 69 forks source link

Adapt voice hints and exporting to changes in BRouter 1.7.X #753

Closed rkflx closed 1 year ago

rkflx commented 1 year ago

Patches contained in this series (see commit messages for details), all relating to adapting to changes in BRouter 1.7.0:

Further changes would be needed (e.g. new Locus format, straight line voice hints, 180° u-turn voice hints at cul-de-sac-style vias, height calculation buffers etc.), but that's all there is for now.

Regarding the new console warning if BRouter is too old, I'm open to better suggestions, of course. I haven't added this new requirement to the CHANGELOG yet, but it should probably be mentioned there and in the release notes once a new release is tagged. Not sure how increasing the required BRouter version was handled in the past.

nrenner commented 1 year ago

Thanks for the PR! I'm fine with merging that as is.

Regarding the new console warning if BRouter is too old, I'm open to better suggestions, of course.

Fine by me.

Not sure how increasing the required BRouter version was handled in the past.

Not at all, other than providing a working standalone bundle with each release. On my to-do list is to include brouter as a dev or peer dependency, which initially wasn't possible because of the GPL license.

rkflx commented 1 year ago

I'll come back to this once the dust in BRouter has settled. We can probably salvage most of it.

include brouter as a dev or peer dependency

That's a nice idea to make it easier to run for new contributors or on a server for sure. It sounds complicated to get working due to the Java-based nature of BRouter though, but admittedly I have not tried yet. Hopefully enough flexibility will be kept to be able to override this with custom BRouter versions (a necessity exemplified by the current situation).

rkflx commented 1 year ago

The reindexing revert in BRouter got in, phew. Since brouter.de/brouter-web seems to be running with or very close to BRouter master, it will be fixed there without needing a new BRouter-Web release.

Hence I reworked this patch series:

nrenner commented 1 year ago

Thanks!