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.

Sign up for GitHub

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

Jump to bottom

performance improvement with manual cholesky mapping #1064

Open
wants to merge 2 commits into
base: branch-24.03
Choose a base branch
from
Open

performance improvement with manual cholesky mapping #1064

wants to merge 2 commits into from

Conversation

rohany
Copy link
Member

@rohany rohany commented Oct 31, 2023 •
edited
Loading

not looking to merge this, but as a followup from the meeting about mapping cholesky better with legate. This change improves the performance by around 1.6x, allowing the legate program to beat cusolvermg.

@rohany rohany requested a review from magnatelee October 31, 2023 19:27
@copy-pr-bot copy-pr-bot
Copy link

copy-pr-bot bot commented Oct 31, 2023

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@marcinz marcinz changed the base branch from branch-23.11 to branch-24.01 November 9, 2023 17:11
@marcinz marcinz changed the base branch from branch-24.01 to branch-24.03 February 22, 2024 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@magnatelee magnatelee Awaiting requested review from magnatelee

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
discussion
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rohany