Skip to content

BUG: Fix test errors steming from the new multi-threading mechanism.#12

Merged
blowekamp merged 1 commit intoSimpleITK:masterfrom
jhlegarreta:FixPoolMultiThreaderErrors
May 29, 2018
Merged

BUG: Fix test errors steming from the new multi-threading mechanism.#12
blowekamp merged 1 commit intoSimpleITK:masterfrom
jhlegarreta:FixPoolMultiThreaderErrors

Conversation

@jhlegarreta
Copy link
Copy Markdown

Fix errors steming from the use of the new itk::PoolMultiThreader
class for multi-threading.

The errors stemmed when this gerrit topic got merged:
http://review.source.kitware.com/#/c/23175/

And were later related to the following gerrit-topic:
http://review.source.kitware.com/#/c/23434/

The solution adopted in this patch set was based on the proposal in:
http://review.source.kitware.com/#/c/23439/

@jhlegarreta jhlegarreta force-pushed the FixPoolMultiThreaderErrors branch from 192b415 to 8f43f00 Compare May 28, 2018 06:29
Fix errors steming from the use of the new `itk::PoolMultiThreader`
class for multi-threading.

The errors stemmed when this gerrit topic got merged:
http://review.source.kitware.com/#/c/23175/

And were later related to the following gerrit-topic:
http://review.source.kitware.com/#/c/23434/

The solution adopted in this patch set was based on the proposal in:
http://review.source.kitware.com/#/c/23439/
@jhlegarreta
Copy link
Copy Markdown
Author

Please merge if appropriate. I do not have write access.

@blowekamp blowekamp merged commit a99bc55 into SimpleITK:master May 29, 2018
@blowekamp
Copy link
Copy Markdown
Member

Thanks for the update!

@jhlegarreta jhlegarreta deleted the FixPoolMultiThreaderErrors branch May 31, 2018 05:43
blowekamp added a commit to blowekamp/ITKSimpleITKFilters that referenced this pull request Nov 2, 2021
Fixed choosing orientation for some previous INVALID conditions with greedy algorithm.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants