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

More extensible layout for cholesky #299

Open
dannys4 opened this issue Jan 5, 2023 · 0 comments
Open

More extensible layout for cholesky #299

dannys4 opened this issue Jan 5, 2023 · 0 comments

Comments

@dannys4
Copy link
Contributor

dannys4 commented Jan 5, 2023

Currently, the cholesky requires you to have LayoutLeft for the sake of CUBLAS requiring it. This is somewhat irritating because the OpenMP backend of kokkos has a default layout of LayoutRight, so when I sample with the Gaussian distribution, I have to call the SampleImpl function directly, lest I want to get a runtime error that says I'm trying to use an incompatible LayoutRight View (i.e. I can't use the default Layout for Views when interacting with the Cholesky, namely within the Gaussian distribution).

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

No branches or pull requests

1 participant