MRtrix3 provides a set of tools to perform various advanced diffusion MRI analyses, including constrained spherical deconvolution (CSD), probabilistic tractography, track-density imaging, and apparent fibre density
Thought that I had resolved this previously, probably when doing #2609, but in doing some other refactoring discovered that the problematic code was still in place. Maybe I fixed it some time but never pushed.
num_exit() was a bad name for the function given its operation; it checks for the number of files in a list that exist. But there was also an integer variable called num_exist.
num_in_use() function immediately got hidden behind integer variable num_in_use.
Function was immediately waiting 60s, rather than the intended operation of using a progressively increasing delay time up to a maximum of 60s.
The functionality should be reviewable from the code, it's pretty obviously wrong. Verifying within an existing script is hard because I can't replicate the circumstances in which it is required, but I've at least tested in an interactive Python terminal.
Thought that I had resolved this previously, probably when doing #2609, but in doing some other refactoring discovered that the problematic code was still in place. Maybe I fixed it some time but never pushed.
num_exit()was a bad name for the function given its operation; it checks for the number of files in a list that exist. But there was also an integer variable callednum_exist.num_in_use()function immediately got hidden behind integer variablenum_in_use.Function was immediately waiting 60s, rather than the intended operation of using a progressively increasing delay time up to a maximum of 60s.
The functionality should be reviewable from the code, it's pretty obviously wrong. Verifying within an existing script is hard because I can't replicate the circumstances in which it is required, but I've at least tested in an interactive Python terminal.