search

linkagescape / linkage-mapper

ArcGIS tools to automate mapping and prioritization of wildlife habitat corridors
https://circuitscape.org/linkagemapper/
GNU General Public License v3.0
39 stars 12 forks source link

Max Number of Connected Neighbors > 4 Can Lead to Crashes #119

Open johngallo opened 3 years ago

johngallo commented 3 years ago

1) make sure the default number of linkages is 4 2) In the help dialog in the tool, indicate that a number higher than 4 may cause crashes, and is only advised if also running linkage priority tool too. 3) consider making the user check a radio box to allow them to do a value higher than 4, and askingthem to read the above disclaimer.

dkav commented 3 years ago

@johngallo are you referring to Step 4 Option A - Maximum Number of Connected Neighbors?

Do you know why the crashes are occurring? It would be helpful to have more detail.

johngallo commented 3 years ago

yes, that is the parameter, thanks.

In some cases, especially with unlimited linkages or 10 linkages, it is a RAM issue. Not sure about this particular case.

On Wed, Mar 17, 2021 at 6:51 PM Darren Kavanagh @.***> wrote:

@johngallo https://github.com/johngallo are you referring to Step 4 Option A - Maximum Number of Connected Neighbors?

Do you know why the crashes are occurring? It would be helpful to have more detail.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/linkagescape/linkage-mapper/issues/119#issuecomment-801555630, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD7CJ2ZHSKZRE7WLJBI2STLTEFMBZANCNFSM4ZK5BMDA .

dkav commented 3 years ago

I am not sure about this. If one runs the tool skipping step 4 will you not run into the same problem? If not, then the issue is with how the prunning is done or how constellations are formed. Without a crash report it is hard to tell.

  • make sure the default number of linkages is 4

The default number is 4.

  • In the help dialog in the tool, indicate that a number higher than 4 may cause crashes, and is only advised if also running linkage priority tool too.

I don't think you should give a specific number or advise for LP only– each user's configuration and needs will differ. You can highlight the implications but that would apply, as far as I understand, to running the tool and skipping step 4.

  • consider making the user check a radio box to allow them to do a value higher than 4, and askingthem to read the above disclaimer.

Radio buttons don't exist for the toolbox, only check boxes. Either way, you are adding more complexity to the tool.

johngallo commented 3 years ago

The low hanging fruit here would be to add the "bomb" icon to the user guide and talk about this there.