function inneighbors does not work on directed weighted graphs

[simonschoelly]

using LightGraphs, SimpleWeightedGraphs
g = SimpleWeightedDiGraph(3)
inneighbors(g, 3)

> ERROR: type Adjoint has no field rowval
Stacktrace:
 [1] getproperty(::Any, ::Symbol) at ./sysimg.jl:18
 [2] inneighbors(::SimpleWeightedDiGraph{Int64,Float64}, ::Int64) at /home/simon/.julia/dev/SimpleWeightedGraphs/src/simpleweighteddigraph.jl:86
 [3] top-level scope at none:0

Using Julia 0.7 (nightly)
outneighbors(g, 3) seems to work though.

[sbromberger]

SimpleWeightedGraphs is going to suffer major breakage soon with 0.7 anyway since our unsafe view will cause segfaults.

[sbromberger]

hm. I can't reproduce this in the latest master:

julia> g = SimpleWeightedDiGraph(3)
{3, 0} directed simple Int64 graph with Float64 weights

julia> inneighbors(g, 3)
0-element Array{Int64,1}

[sbromberger]

@simonschoelly - can you test the latest master on 1.0 for performance? I want to make sure we don't have any regressions.

[simonschoelly]

I think the reason for that error was, that you had code of the form

mat  = g.weights'
mat.rowval

which does now work because mat is of type Adjoint. Now you are doing

mat = SparseMatrixCSC(g.weight')
mat.rowval

which does work. The tests for the latest master pass on 1.0. Is that what you mean by performance?

[sbromberger]

I'm going to close this out because it's fixed in master.