Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --workers option for multi-process operations #675

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dirtyhillbilly
Copy link

This MR allows checking files in a ProcessPoolExecutor
With my test base (~6500 files), and quad-core i5 cpu, --workers 0 is twice as fast as sequential operations.

@coveralls
Copy link

Coverage Status

coverage: 99.476% (-0.3%) from 99.825%
when pulling 922347d on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

@coveralls
Copy link

Coverage Status

coverage: 99.476% (-0.3%) from 99.825%
when pulling 5f88109 on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

@coveralls
Copy link

Coverage Status

coverage: 99.651% (-0.2%) from 99.825%
when pulling 9976bff on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

@coveralls
Copy link

Coverage Status

coverage: 99.677% (-0.1%) from 99.825%
when pulling b321c52 on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

@coveralls
Copy link

Coverage Status

coverage: 99.677% (-0.1%) from 99.825%
when pulling e1e0813 on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

1 similar comment
@coveralls
Copy link

Coverage Status

coverage: 99.677% (-0.1%) from 99.825%
when pulling e1e0813 on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

@dirtyhillbilly dirtyhillbilly force-pushed the feat-processpoolexecutor branch 2 times, most recently from ce945f4 to 41e34b8 Compare June 7, 2024 09:42
@coveralls
Copy link

Coverage Status

coverage: 99.752% (-0.07%) from 99.825%
when pulling 41e34b8 on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

@coveralls
Copy link

Coverage Status

coverage: 99.827% (+0.002%) from 99.825%
when pulling 1381c0b on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

1 similar comment
@coveralls
Copy link

Coverage Status

coverage: 99.827% (+0.002%) from 99.825%
when pulling 1381c0b on dirtyhillbilly:feat-processpoolexecutor
into 81e9f98 on adrienverge:master.

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.

2 participants