route_graphhopper is out of date

[Robinlovelace]

I think a package could be the best approach to keep it modular. Improvements on the current function could include:

• use of their updated API
• return of per-segment rather than per-route data
• use of sf

After a bit of searching, I've realised that there's no need to, @crazycapivara has already created a modern interface that returns sf objects 🎉 Great work!

The only remaining issue from this package's perspective are to:

• Deprecate route_graphhopper()
• Test the use of route() with graphhopper::gh_get_route() and document as appropriate

[Robinlovelace]

Link to the graphopper repo: https://github.com/crazycapivara/graphhopper-r

[crazycapivara]

@Robinlovelace Good to know that you want to use the lib. I hope I can publish it soon on cran.

[crazycapivara]

crazycapivara/graphhopper-r#8

[Robinlovelace]

Great, let me know if you want a hand with that. Looking good to me!

[Robinlovelace]

Test the use of route() with graphhopper::gh_get_route() and document as appropriate

Still not done but optional extra so closing for now.

[crazycapivara]

graphhopper is now on CRAN