-
Notifications
You must be signed in to change notification settings - Fork 577
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
Kokkos/Kokkos-Kernels: support view of crs matrices on cuda #2648
Conversation
These changes are already in the develop branches of the kokkos and kokkos-kernels repos. We need them in Trilinos now so that we can merge changes in panzer that require a view of crs matrices.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good, match changes approved by commits to Kokkos and KokkosKernels develop branches, which will clobber this commit during the next snapshot.
See PRs kokkos/kokkos#1587 and kokkos/kokkos-kernels#220 |
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request. |
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
Using Repos:
Pull Request Author: rppawlo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @rppawlo ! :D
Thanks @mhoemmen for your help on the nested views! |
Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand)Build InformationTest Name: Trilinos_pullrequest_gcc_4.9.3
Jenkins Parameters
Build InformationTest Name: Trilinos_pullrequest_gcc_4.8.4
Jenkins Parameters
|
Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ ndellingwood mhoemmen ]! |
Status Flag 'Pull Request AutoTester' - Pull Request MUST BE MERGED MANUALLY BY Project Team - Master Automerge is disabled (in .cfg file) |
These are chagnes to support a view of crs matrices on cuda. The same changes have already been accepted into the kokkos and kokkos-kernels repositories. So any new snapshop will retain these changes. We need them added before the next kokkos sync to support work in panzer.
@trilinos/kokkos
@trilinos/kokkos-kernels
@trilinos/panzer
Description
Motivation and Context
How Has This Been Tested?
Checklist