Skip to content

Use Cholesky decomposition for a more stable LLPR#1043

Open
frostedoyster wants to merge 3 commits intomainfrom
llpr-cholesky
Open

Use Cholesky decomposition for a more stable LLPR#1043
frostedoyster wants to merge 3 commits intomainfrom
llpr-cholesky

Conversation

@frostedoyster
Copy link
Collaborator

@frostedoyster frostedoyster commented Feb 12, 2026

Contributor (creator of pull-request) checklist

  • Tests updated (for new features and bugfixes)?
  • Documentation updated (for new features)?
  • Issue referenced (for PRs that solve an issue)?

Maintainer/Reviewer checklist

  • CHANGELOG updated with public API or any other important changes?
  • GPU tests passed (maintainer comment: "cscs-ci run")?

📚 Documentation preview 📚: https://metatrain--1043.org.readthedocs.build/en/1043/

# normal distribution
ensemble_displacements = (
torch.linalg.solve_triangular(
cur_cholesky.T,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we avoid transposing and passing upper=False?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not the same as far as I understand

@frostedoyster
Copy link
Collaborator Author

cscs-ci run

3 similar comments
@frostedoyster
Copy link
Collaborator Author

cscs-ci run

@ppegolo
Copy link
Contributor

ppegolo commented Feb 12, 2026

cscs-ci run

@frostedoyster
Copy link
Collaborator Author

cscs-ci run

@frostedoyster frostedoyster enabled auto-merge (squash) February 13, 2026 03:21
@frostedoyster
Copy link
Collaborator Author

cscs-ci run

2 similar comments
@frostedoyster
Copy link
Collaborator Author

cscs-ci run

@frostedoyster
Copy link
Collaborator Author

cscs-ci run

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.

2 participants