LightOSM: precomputing dijkstra_states throws an UndefVarError

From LightOSM.jl/benchmarks.jl at master · DeloitteOptimalReality/LightOSM.jl · GitHub

point = GeoLocation(-37.8142176, 144.9631608) # Melbourne, Australia
radius = 5 # km
data_file = joinpath(@__DIR__, "benchmark_map.osm")

g_losm = LightOSM.graph_from_download(:point,
                                      point=point,
                                      radius=radius,
                                      weight_type=:distance,
                                      download_format=:osm,
                                      save_to_file_location=data_file)

# g_losm = LightOSM.graph_from_file(data_file, weight_type=:distance)
@time g_losm_precompute = LightOSM.graph_from_file(data_file, weight_type=:distance, precompute_dijkstra_states=true)

throws and UndefVarError blaming a parameter n in src/graph.jl:455 which simply initialises a vector to hold g.dijkstra_states = Vector{Vector{U}}(undef,n)
Shouldn’t this be g.dijkstra_states = Vector{Vector{U}}(undef,length(g.nodes)) ?

I think you’re right. I’d click the edit button on LightOSM.jl/graph.jl at 33898b9b45728878546ac22775347e7ae2459dd2 · DeloitteOptimalReality/LightOSM.jl · GitHub and propose the change :slight_smile:

1 Like