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

Drop multithreading in bootstrap #675

Open
palday opened this issue Apr 11, 2023 · 0 comments
Open

Drop multithreading in bootstrap #675

palday opened this issue Apr 11, 2023 · 0 comments

Comments

@palday
Copy link
Member

palday commented Apr 11, 2023

With improvements/changes in BLAS multithreading, Julia-level threads are competing with the BLAS threads and we don't see any improvement in most situations with use_threads=true. In fact, we sometimes see performance degradation. So let's drop multithreading for the bootstrap and simplify that code. We should instead develop an example for the docs showing how to optionally parallelize using Distributed for people with access to clusters / large computer.

NB: This doesn't prevent us from adding threading to various custom linear algebra methods, if those prove worthwhile. That said, I think that should be an internal performance optimization and not expose any changes in the external API.

@palday palday added this to the MixedModels 5.0 milestone Apr 11, 2023
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

No branches or pull requests

1 participant