Closed saimarathore closed 4 years ago
@AlexanderGetka-cbica could you please ensure that this PR is tested before we merge (as it is touching upon multiple applications)?
Alright, I will ping the dev team once the artifacts have been tested.
@AlexanderGetka-cbica: no idea if you have come across this yet, but one mechanism you could possibly use to write the range-based unit test for training module is via CMake itself [ref]:
foreach(<loop_var> RANGE <start> <stop> [<step>])
@AlexanderGetka-cbica: no idea if you have come across this yet, but one mechanism you could possibly use to write the range-based unit test for training module is via CMake itself [ref]:
foreach(<loop_var> RANGE <start> <stop> [<step>])
Testing is complete and this is good to merge. Unit testing will be part of a separate PR.
training module options fixed from GUI and CLI