Copliot brought to my attention that vpr_autoid_read() has a very high cyclomatic complexity of 23
It should be reduced by removing some of the conditional (if) statements. I think this issue is mainly due to the inclusion of aid and aidmeas data within the same function. Now that we have moved away from Visual Plankton we could split this into two functions, one to read measurement data (from ImageJ) and one to read aid files. Really the measurement data workflow has been separated from vprr in practicality. Maybe it could be removed all together.
Copliot brought to my attention that
vpr_autoid_read()has a very high cyclomatic complexity of 23It should be reduced by removing some of the conditional (if) statements. I think this issue is mainly due to the inclusion of aid and aidmeas data within the same function. Now that we have moved away from Visual Plankton we could split this into two functions, one to read measurement data (from ImageJ) and one to read aid files. Really the measurement data workflow has been separated from vprr in practicality. Maybe it could be removed all together.
@kevinsorochan thoughts?