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

Make SuiteSparse wrapper thread safe. #34546

Merged
merged 1 commit into from
Jan 31, 2020
Merged

Conversation

andreasnoack
Copy link
Member

@andreasnoack andreasnoack commented Jan 28, 2020

By storing an instance of the cholmod_common struct per thread. We might be able to utilize a similar approach to support Int32 indices.

Fixes #34500

@andreasnoack andreasnoack added backport 1.0 kind:bugfix This change fixes an existing bug domain:linear algebra Linear algebra domain:multithreading Base.Threads and related functionality domain:arrays:sparse Sparse arrays labels Jan 28, 2020
@KristofferC KristofferC merged commit 51f1710 into master Jan 31, 2020
@KristofferC KristofferC deleted the an/suitesparsethreads branch January 31, 2020 07:16
KristofferC pushed a commit that referenced this pull request Jan 31, 2020
@KristofferC KristofferC mentioned this pull request Jan 31, 2020
26 tasks
@KristofferC KristofferC mentioned this pull request Feb 20, 2020
56 tasks
mtanneau added a commit to ds4dm/Tulip.jl that referenced this pull request Feb 29, 2020
Julia version is restricted to 1.3 until modifications in CHOLMOD brought by JuliaLang/julia#34546 are addressed.
KristofferC pushed a commit that referenced this pull request Apr 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:arrays:sparse Sparse arrays domain:linear algebra Linear algebra domain:multithreading Base.Threads and related functionality kind:bugfix This change fixes an existing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Segfault in factorizations inside threaded loops
2 participants