search

sbromberger / SimpleWeightedGraphs.jl

Simple weighted graphs. Requires LightGraphs.jl.
Other
49 stars 22 forks source link

Update simpleweighteddigraph.jl #68

Closed sbromberger closed 3 years ago

sbromberger commented 3 years ago

Fixes https://github.com/JuliaGraphs/LightGraphs.jl/issues/1523 and #67.

gdalle commented 3 years ago

@sbromberger judging by https://github.com/JuliaGraphs/SimpleWeightedGraphs.jl/blob/1df10ced7be9ab204a1a23d9cde5470d4acccf55/src/simpleweighteddigraph.jl#L86 wouldn't it be better to replace g.weights' by weights(g) in our fix? Not very important I guess but maybe more Julianic