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

Tpetra: Distributed dense matrix type with Layout right #13565

Open
Adrian-Diaz opened this issue Nov 1, 2024 · 4 comments
Open

Tpetra: Distributed dense matrix type with Layout right #13565

Adrian-Diaz opened this issue Nov 1, 2024 · 4 comments

Comments

@Adrian-Diaz
Copy link

@trilinos/tpetra

Greetings Everyone,

Is there a Tpetra type (or other package with a similar distributed type) that can serve as a dense matrix using Layout Right? In theory the multivector type could work but it seems to me thats hard coded to always use Layout Left. I also saw CRS Matrix which allows Layout Right with a 1D storage view, but that class seems to carry the overhead of the graph matrix for a dense application. Thanks in advance for any additional info on the matter.

Adrian Diaz

@cgcgcg
Copy link
Contributor

cgcgcg commented Nov 4, 2024

@trilinos/tpetra

@csiefer2
Copy link
Member

csiefer2 commented Nov 5, 2024

You are correct. MultiVector is hardcoded to LayoutLeft. AFAIK, we don't have a LayoutRight distributed object.

@Adrian-Diaz
Copy link
Author

Hmmm, would anything break if Multivector or a renamed copy of it was compiled with layout right for that argument? At least anything major that comes to mind, naturally there's lots of details and it may not even compile.

@csiefer2
Copy link
Member

csiefer2 commented Nov 11, 2024

Vector extraction would break in a "LayoutRightMultiVector"... that'll break some of the Kyrlov solvers, for instance.

That said, if there's a strong driver for a layout right multivector, we could definitely consider adding one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants