Skip to content

Add limit for n_jobs with a max of os.cpu_count()#2940

Merged
samuelgarcia merged 5 commits intoSpikeInterface:mainfrom
zm711:n-jobs-limit-proposal
Jun 5, 2024
Merged

Add limit for n_jobs with a max of os.cpu_count()#2940
samuelgarcia merged 5 commits intoSpikeInterface:mainfrom
zm711:n-jobs-limit-proposal

Conversation

@zm711
Copy link
Copy Markdown
Member

@zm711 zm711 commented May 30, 2024

Based on discussion in #2875.

@samuelgarcia still needs to weigh in. But this is just the proposal so can be closed if we don't want this.

@zm711
Copy link
Copy Markdown
Member Author

zm711 commented May 30, 2024

Before I fix the tests, I'll wait for Sam to weigh in, but if @alejoe91 or @h-mayorquin have changed their minds feel free to comment.

@zm711 zm711 added the core Changes to core module label May 31, 2024
@zm711 zm711 marked this pull request as ready for review May 31, 2024 13:21
@alejoe91 alejoe91 added the hackathon-24 Contributions during the SpikeInterface Hackathon May 24 label Jun 1, 2024
@alejoe91
Copy link
Copy Markdown
Member

alejoe91 commented Jun 1, 2024

Parfait pour moi!

@samuelgarcia
Copy link
Copy Markdown
Member

Merci.

@samuelgarcia samuelgarcia merged commit bd59976 into SpikeInterface:main Jun 5, 2024
@alejoe91
Copy link
Copy Markdown
Member

alejoe91 commented Jun 5, 2024

Grazie Zach :)

@zm711 zm711 deleted the n-jobs-limit-proposal branch April 25, 2025 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Changes to core module hackathon-24 Contributions during the SpikeInterface Hackathon May 24

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants