Skip to content

Change to the Modified Gram Schmidt algorithm in orthogonalise (#679)#785

Merged
mscroggs merged 1 commit intoFEniCS:mainfrom
ampdes:update-orthogonalise-mgs
Feb 9, 2024
Merged

Change to the Modified Gram Schmidt algorithm in orthogonalise (#679)#785
mscroggs merged 1 commit intoFEniCS:mainfrom
ampdes:update-orthogonalise-mgs

Conversation

@ampdes
Copy link
Contributor

@ampdes ampdes commented Feb 9, 2024

  • change the orthogonalization algorithm to modified gram schmidt

  • norm is precomputed

  • first check row rank before orthogonalizing the rows

@jhale
Copy link
Member

jhale commented Feb 9, 2024

Looks good; please run clang-format -i on the file.

@ampdes ampdes force-pushed the update-orthogonalise-mgs branch from c49b1be to 04eb676 Compare February 9, 2024 17:45
…S#679)

* change the orthogonalization algorithm to modified gram schmidt

* norm is precomputed

* first check row rank before orthogonalizing the rows
@ampdes ampdes force-pushed the update-orthogonalise-mgs branch from 04eb676 to e6d92f7 Compare February 9, 2024 17:49
@mscroggs mscroggs linked an issue Feb 9, 2024 that may be closed by this pull request
@mscroggs mscroggs added this pull request to the merge queue Feb 9, 2024
Merged via the queue into FEniCS:main with commit 747e66f Feb 9, 2024
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.

Replace Gram-Schmidt orthogonalisation with better algorithm

3 participants