Not sure what changed so it might worth a bug report/doc update but
can you try this?
function MinCutVisitor{T}(graph::SimpleGraph, distmx::AbstractArray{T, 2})
n = nv(graph)
MinCutVisitor{T}(
graph,
falses(n),
zeros(Int,n),
typemax(T),
zero(T),
zero(Int),
distmx,
@compat(Vector{Int}())
)
end
I.e. add `{T}` to the call to the default constructor
On Tue, Jul 7, 2015 at 8:05 PM, andrew cooke <[email protected]> wrote:
>
> what's T in the last chunk of code? you have typemax(T), but no T as a type
> parameter. is that really working?
>
>
> On Tuesday, 7 July 2015 20:03:24 UTC-3, Seth wrote:
>>
>> I have the following code:
>>
>> type MinCutVisitor{T} <: AbstractMASVisitor
>> graph::SimpleGraph
>> parities::AbstractArray{Bool,1}
>> colormap::Vector{Int}
>> bestweight::T
>> cutweight::T
>> visited::Integer
>> distmx::AbstractArray{T, 2}
>> vertices::Vector{Int}
>> end
>>
>> function MinCutVisitor{T}(graph::SimpleGraph, distmx::AbstractArray{T, 2})
>> n = nv(graph)
>> MinCutVisitor(
>> graph,
>> falses(n),
>> zeros(Int,n),
>> typemax(T),
>> zero(T),
>> zero(Int),
>> distmx,
>> @compat(Vector{Int}())
>> )
>> end
>>
>>
>> and up until June 30th the outer constructor was working in 0.4:
>>
>> julia> g = Graph(8)
>> {8, 0} undirected graph
>>
>>
>> julia> LightGraphs.MinCutVisitor(g,spzeros(Float64,8,8))
>> LightGraphs.MinCutVisitor{Float64}({8, 0} undirected
>> graph,Bool[false,false,false,false,false,false,false,false],[0,0,0,0,0,0,0,0],Inf,0.0,0,8x8
>> sparse matrix with 0 Float64 entries:,Int64[])
>>
>>
>> Now it fails on Float64 but not with Int:
>>
>> julia> LightGraphs.MinCutVisitor(g,spzeros(Float64,8,8))
>> ERROR: MethodError: `convert` has no method matching
>> convert(::Type{LightGraphs.MinCutVisitor{T}}, ::LightGraphs.Graph,
>> ::BitArray{1}, ::Array{Int64,1}, ::Float64, ::Float64, ::Int64,
>> ::Base.SparseMatrix.SparseMatrixCSC{Float64,Int64}, ::Array{Int64,1})
>> This may have arisen from a call to the constructor
>> LightGraphs.MinCutVisitor{T}(...),
>> since type constructors fall back to convert methods.
>> Closest candidates are:
>>
>> LightGraphs.MinCutVisitor{T}(::Union{LightGraphs.Graph,LightGraphs.DiGraph},
>> ::AbstractArray{Bool,1}, ::Array{Int64,1}, ::T, ::T, ::Integer,
>> ::AbstractArray{T,2}, ::Array{Int64,1})
>>
>> LightGraphs.MinCutVisitor{T}(::Union{LightGraphs.Graph,LightGraphs.DiGraph},
>> ::AbstractArray{T,2})
>> call{T}(::Type{T}, ::Any)
>> ...
>> in call at /Users/seth/.julia/v0.4/LightGraphs/src/maxadjvisit.jl:99
>>
>>
>> julia> LightGraphs.MinCutVisitor(g,spzeros(Int,8,8))
>> LightGraphs.MinCutVisitor{Int64}({8, 0} undirected
>> graph,Bool[false,false,false,false,false,false,false,false],[0,0,0,0,0,0,0,0],9223372036854775807,0,0,8x8
>> sparse matrix with 0 Int64 entries:,Int64[])
>>
>> However, if I remove the type parameterization from the outer constructor:
>>
>> function MinCutVisitor(graph::SimpleGraph, distmx::AbstractArray)
>>
>> n = nv(graph)
>> MinCutVisitor(
>> graph,
>> falses(n),
>> zeros(Int,n),
>> typemax(T),
>> zero(T),
>> zero(Int),
>> distmx,
>> @compat(Vector{Int}())
>> )
>> end
>>
>>
>> it works. Can someone tell me why this is?
