Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
mariesoret opened this issue Oct 18, 2021 · 8 comments

Comments

@mariesoret
Copy link

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
Copy link
Member

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

@ranjanan
Copy link
Member

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

@mariesoret
Copy link
Author

Sure, when do you plan to release the update ?

@ranjanan
Copy link
Member

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

@mariesoret
Copy link
Author

mariesoret commented Oct 18, 2021

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

@mariesoret
Copy link
Author

The job seemed to run fine 👏 👏 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
Copy link
Member

vlandau commented Oct 18, 2021

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

@ranjanan
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants