search

Circuitscape / Circuitscape.jl

Algorithms from circuit theory to predict connectivity in heterogeneous landscapes
https://circuitscape.org
MIT License
128 stars 35 forks source link

Test failed on Debian : no method matching connected_components(::SimpleWeightedGraphs.SimpleWeightedGraph{Int64, Float64}) #321

Closed mariesoret closed 3 years ago

mariesoret commented 3 years ago

Every simulations I tried is ending with an error like this one :

MethodError: no method matching connected_components(::SimpleWeightedGraphs.SimpleWeightedGraph{Int64, Float64})
  Closest candidates are:
    connected_components(::LightGraphs.AbstractGraph{T}) where T at /home/msoret/.julia/packages/LightGraphs/IgJif/src/connectivity.jl:100

So I tried to run Pkg.test("Circuitscape") and saw the same error several times. Here is the full stacktrace.

I'm using Circuitscape v5.8.4 with Julia v1.6.3 on Debian 10. I also tried with Circuitscape 5.7.1 and Julia 1.5.3.

ranjanan commented 3 years ago

This is fixed on master. Let me release a version.

ranjanan commented 3 years ago

Once this is merged , could you please update to 5.9.0 and try?

mariesoret commented 3 years ago

Sure, when do you plan to release the update ?

ranjanan commented 3 years ago

Once https://github.com/JuliaRegistries/General/pull/46989 merges, the update should available for you to download. You'll have to run using Pkg; Pkg.update() after that.

mariesoret commented 3 years ago

thanks a lot for your reactivity! I'm currently trying the master branch, I'll keep you informed

mariesoret commented 3 years ago

The job seemed to run fine :clap: :clap: However I had to kill it for lack of disk place.

I have over 23000 focal nodes and each output represents 134 MB.

Is there a way to reduce the size of asc output files? Or to choose another output format such as geotiff ? Maybe I should address this in a dedicated issue.

vlandau commented 3 years ago

@mariesoret using GeoTiff will save some space as compared to ASCII. Just set write_as_tif = true in the INI file

ranjanan commented 3 years ago

I am closing this issue because the original issue was resolve. @mariesoret glad to hear it worked.