UrbanAnalyst / dodgr

Distances on Directed Graphs in R
https://urbananalyst.github.io/dodgr/
127 stars 16 forks source link

Default to geodesic distances #179

Closed mpadge closed 2 years ago

mpadge commented 2 years ago

Distance calculations for sc currently use default geodist method, which is "cheap" distance, while for sf they use an internally-coded Haversine calculation. These can both be ditched, and a distance column inserted after return from the C++ routines with geodist(..., measure = "geodesic"). The extra calculation time for geodesic distances is still only a tiny fraction of total graph construction time, so can easily be implemented with hardly any slowdown. Prompted by comments of @mguzmann in #178.