AddNodeStrategy.CreateChildGenome() has an unhandled case that results in an IndexOutOfRangeException

[joelmeaders]

I'm running into a case where insertIdx1 is resolved to -1, which leads to stopIdx, copyLen and parentIdx being -1. IndexOutOfRangeException is thrown at connArr[childIdx] = newConnArr[newConIdx];

newConnArr: 0,14 8,0

newWeightArr: -0.4854858 1

parentConnArr: 0,14 4,14 8,14 13,14

parentWeightArr: -4.0975833 -3.7439632 -0.4854858 0.24271071

[colgreen]

Hi. Was this on main branch? And have you seen this more than once?

[joelmeaders]

This was on the main branch.

I may have tracked down the cause. I was reusing AddNodeStrategy rather than creating a new one for subsequent generations so the original sequence IDs were being reused.

[colgreen]

At first glance it does seem that method doesn't handle when insertIdx1 is -1, so it's a bit odd that I've never seen this issue. It's quite a complex method and it's late here, so I'll need to take a fresh look at it another time. I feel like this method should be simplified and that I over optimised it perhaps.