ENH: Add NumberOfThreads property to itk::TransformixFilter#714
ENH: Add NumberOfThreads property to itk::TransformixFilter#714
NumberOfThreads property to itk::TransformixFilter#714Conversation
ffab4b9 to
c3dd02c
Compare
Addressed issue InsightSoftwareConsortium/ITKElastix#163 "No option to set number of threads for Transformix", submitted by Sebastian van der Voort.
c3dd02c to
96df5ee
Compare
|
Sorry, after submitting this pull request, I started doubting. If |
|
@N-Dekker There is a per-instance SetNumberOfWorkUnits on all |
@thewtex Thanks for the suggestion Matt. I did give it an initial try before, with https://github.com/SuperElastix/elastix/compare/NumberOfWorkUnits It sets the NumberOfWorkUnits for each elastix component that is derived from itk::ProcessObject. However, I have not finished this approach, because it does not really limit the number of threads used by each individual component. The NumberOfWorkUnits is not (yet?) passed to each sub-object of each component. I don't know if it's already worth merging this incomplete commit. If users really want to reduce the number of threads, they should just call |
|
@N-Dekker that's great, it looks like with some more work to propagate NumberOfWorkUnits to the sub-objects developers will be able to request the number of threads used on a per-filter basis. |
Addressed issue InsightSoftwareConsortium/ITKElastix#163 "No option to set number of threads for Transformix", submitted by Sebastian van der Voort (@Svdvoort).