-
Notifications
You must be signed in to change notification settings - Fork 221
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
Issue #2533: Preserving the buffer type in linear algebra #2534
base: master
Are you sure you want to change the base?
Conversation
Thanks for the PR! It is often easier to use the
|
@maleadt Oh yes, similar is a better way to go. I will update these changes with similar, thanks! |
Yeah looks better with |
…o kmp5/debug/issue_2533
…o kmp5/debug/issue_2533
LGTM, CI failure unrelated. Why the partial revert though? |
@maleadt I wasn't sure what the failures were related to. With the changes I reverted, I haven't yet had time to verify that they work as expected so I wasn't sure if those changes were causing the check failures. I have more time today and can check that function and re-push the changes. |
@maleadt I was able to check that the gebrd! code does work properly if all the tensors are on unified memory. I added back those changes. Thanks! |
An attempt to fix buffer type linear algebra issue. It looks like the
CuArray
function is called many times and there is no book keeping of thememory_type
. I manually added to theCuArray
function calls. One could also make this part of the function call. See Issue #2533