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

Improve batched serial trsv #2452

Merged
merged 8 commits into from
Dec 11, 2024

Conversation

yasahi-hpc
Copy link
Contributor

This PR aims at improving the implementation and testing of serial Trsv.

  • Moving implementation details into Impl namespace.
  • Add all the specializations of the combinations of Upper/Lower, Non-Trans/Trans/ConjTrans, and Unit/Non-Unit
  • Add a check function of input to force A is rank 2 and b is rank 1 Kokkos::View
  • Covering all the unit-tests for all the combinations of Upper/Lower, Non-Trans/Trans/ConjTrans, and Unit/Non-Unit

As a TO DO task, we need to add a ConjTrans implementation of blocked version which requires a little more investigation

@yasahi-hpc yasahi-hpc force-pushed the improve-batched-serial-trsv branch from f16925c to 392745c Compare December 6, 2024 13:41
@lucbv lucbv requested review from brian-kelley and lucbv December 6, 2024 17:14
Yuuichi Asahi added 8 commits December 10, 2024 13:10
Signed-off-by: Yuuichi Asahi <[email protected]>
Signed-off-by: Yuuichi Asahi <[email protected]>
Signed-off-by: Yuuichi Asahi <[email protected]>
Signed-off-by: Yuuichi Asahi <[email protected]>
Signed-off-by: Yuuichi Asahi <[email protected]>
@yasahi-hpc yasahi-hpc force-pushed the improve-batched-serial-trsv branch from 07ad22e to 78a2066 Compare December 10, 2024 04:12
@yasahi-hpc
Copy link
Contributor Author

@brian-kelley
Hi, I have fixed accordingly.
May I have another review please

@brian-kelley brian-kelley added AT: PRE-TEST INSPECTED Mark this PR as approved for testing. AT2-CI-APPROVAL Approve CI to run at SNL labels Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT: PRE-TEST INSPECTED Mark this PR as approved for testing. AT2-CI-APPROVAL Approve CI to run at SNL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants