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 cholmod preserve user-specified permutation #40560

Merged
merged 3 commits into from
Apr 28, 2021

Conversation

ettersi
Copy link
Contributor

@ettersi ettersi commented Apr 22, 2021

@dkarrasch dkarrasch added linear algebra Linear algebra sparse Sparse arrays labels Apr 22, 2021
@ettersi ettersi changed the title Make cholmod preserve user-specified permutation [WIP] Make cholmod preserve user-specified permutation Apr 22, 2021
@ettersi ettersi marked this pull request as draft April 23, 2021 03:40
@ettersi ettersi force-pushed the cholmod branch 2 times, most recently from 6d1fae8 to fa9ed7f Compare April 23, 2021 10:23
@ettersi
Copy link
Contributor Author

ettersi commented Apr 23, 2021

Okay, now it seems to be working. Global state was taking me for a ride the first time around, so I changed the entire code base such that all changes to the common object are only temporary. I also added a test to make sure that all (relevant) parts of the common object agree with their defaults.

@ettersi ettersi marked this pull request as ready for review April 23, 2021 10:25
@ettersi ettersi changed the title [WIP] Make cholmod preserve user-specified permutation Make cholmod preserve user-specified permutation Apr 23, 2021
@ViralBShah
Copy link
Member

Yikes, sorry about that conflict. Can you rebase? Also good to get a few more eyes on this. @andreasnoack perhaps.

@ettersi
Copy link
Contributor Author

ettersi commented Apr 26, 2021

Rebase done!

@ViralBShah ViralBShah merged commit 907264b into JuliaLang:master Apr 28, 2021
@ettersi ettersi deleted the cholmod branch April 28, 2021 00:23
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
* Make cholmod preserve user-specified permutation

* Ensure all changes to CHOLMOD.common are temporary

* Fix spurious comment

Co-authored-by: Simon Etter <ettersi@nus.edu.sg>
jarlebring pushed a commit to jarlebring/julia that referenced this pull request May 4, 2021
* Make cholmod preserve user-specified permutation

* Ensure all changes to CHOLMOD.common are temporary

* Fix spurious comment

Co-authored-by: Simon Etter <ettersi@nus.edu.sg>
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
* Make cholmod preserve user-specified permutation

* Ensure all changes to CHOLMOD.common are temporary

* Fix spurious comment

Co-authored-by: Simon Etter <ettersi@nus.edu.sg>
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
* Make cholmod preserve user-specified permutation

* Ensure all changes to CHOLMOD.common are temporary

* Fix spurious comment

Co-authored-by: Simon Etter <ettersi@nus.edu.sg>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linear algebra Linear algebra sparse Sparse arrays
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug in sparse cholesky permutation argument
3 participants