Max Number of Connected Neighbors > 4 Can Lead to Crashes

[johngallo]

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]

@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]

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 <#119 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AD7CJ2ZHSKZRE7WLJBI2STLTEFMBZANCNFSM4ZK5BMDA> .

[dkav]

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]

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