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

Circuitscape 5 is not writing cumulative branch and node current files for network mode #303

Closed
venkanna37 opened this issue Jul 6, 2021 · 11 comments
Assignees

Comments

@venkanna37
Copy link

venkanna37 commented Jul 6, 2021

Circuitscape 5 is not writing cumulative currents files when I run with the network data type. While the same settings (.ini and habitat files) are working fine with Circuitscape 4 and are giving the desired cumulative branch and node currents files.

I am running Circuitscape 5 (v5.8.1) in Julia v1.6.1 with the following .ini and network data:
Circuitscape_graph2.txt
Circuitscape_network_ini.txt

Tagging @indranil-wii

@vlandau
Copy link
Member

vlandau commented Aug 10, 2021

Hi @venkanna37, apologies for such a late response to this issue. I think this must be because you are providing the same file as both your point file and your habitat map. For network and raster data in pariwise mode, you need to supply a separate file that specifies the points (nodes) that you want to connect.

From the docs:

For pairwise analysis we would also supply a list of focal nodes (containing at least two node numbers, but as many as...the number of nodes in the circuit) between which we want to perform calculations.

@venkanna37
Copy link
Author

Hello @vlandau , thank you for the reply and apologies the for late response. After modifying the point file, the branch and node current files are improved, the resistance column are not being considered as nodes (before modification, resistances were considered as nodes in output). Still, Circuitescape5 is not writing cumulative node and branch currents files. I am attaching my .ini, point and resistance files, can you please check when you find the time and let me know if I have to make any modification in them.

Thank you.

Circuitscape_network_ini.txt
centrality_points.txt
centrality_resistances.txt

@vlandau
Copy link
Member

vlandau commented Sep 16, 2021

Unfortunately for network mode I think the cumulative branch currents are currently not able to be written, see #119. cc @ranjanan

@venkanna37 venkanna37 changed the title Circuitscape 5 is not writing cumulative current files Circuitscape 5 is not writing cumulative branch and node current files for network mode Sep 17, 2021
@venkanna37
Copy link
Author

Thank you for confirmation @vlandau. It will be very useful if Circuitescape-5 can write cumulative branch and node current files.

@johngallo
Copy link

Hi @ViralBShah, @ranjanan , and @vlandau ,

Thank you for the creation of circuitscape.jl and ominiscape.jl .

Posting here rather than email on the slight chance there is someone out there that is already working on this fix, and they can notify here with an update or a PR.

I've been briefed by @venkanna37 and my understanding is that this Issue the last bit of functionality that needs to be introduced to circuitscape.jl to complete the project of replicating all the necessary functionality of circuitscape.py. A bonus of this occurring is that we will be able to introduce the Linkage Mapper community to circuitscape.jl as the means for performing Pinchpoint Mapper and Centrality Mapper, rather than using circuitscape.py.

Given this context, please let us know what your plans are for this Issue.

Thank you,

John

@ViralBShah
Copy link
Member

@ranjanan I presume this is straightforward to do at some point. Do you know when you might find the time?

@ranjanan
Copy link
Member

ranjanan commented Oct 12, 2021

This should be straightforward. I will find time within this week or the next.

@johngallo
Copy link

Wonderful. Thanks Ranjan and good luck!

@ranjanan
Copy link
Member

Closed by #318

@ranjanan
Copy link
Member

ranjanan commented Oct 20, 2021

@venkanna37 and @johngallo this should now work with version 5.9.0

@venkanna37
Copy link
Author

Thank you so much @ranjanan I will try it

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

5 participants