Skip to content

Validate ThreadContext.num_workers#3

Open
cydanil wants to merge 1 commit intophilsmt:masterfrom
cydanil:feat/handle_zero
Open

Validate ThreadContext.num_workers#3
cydanil wants to merge 1 commit intophilsmt:masterfrom
cydanil:feat/handle_zero

Conversation

@cydanil
Copy link

@cydanil cydanil commented Nov 10, 2021

Hi @philsmt,

Here is a possible improvement validating the given ThreadContext.num_workers at initialization.

Messing around with AGIPD notebooks, I noticed that the notebook may assign 0 threads if there's more files than cores availables.

It's most likely a non-issue but it's a slow shift it's a simple validation to add here.

I've tested the changes as seen in the test file.

Thanks!
Cyril

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.

1 participant