-
Notifications
You must be signed in to change notification settings - Fork 2
Lesson 10: Launch - Matrix transpose #42
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
Conversation
Intro_Tutorial/lessons/12_raja_device_kernel_complete/12_raja_launch_transpose.cpp
Outdated
Show resolved
Hide resolved
|
We need a companion readme here |
...torial/lessons/12_raja_device_kernel_complete/solution/12_raja_launch_transpose_solution.cpp
Outdated
Show resolved
Hide resolved
...torial/lessons/12_raja_device_kernel_complete/solution/12_raja_launch_transpose_solution.cpp
Outdated
Show resolved
Hide resolved
|
Let's rename this lesson 10. |
Intro_Tutorial/lessons/12_raja_device_kernel_complete/12_raja_launch_transpose.cpp
Outdated
Show resolved
Hide resolved
|
Won't be an exercise for the participants, we will just explain what is going on. |
|
Hi @kab163 @johnbowen42 @rhornung67 @rchen20 , I think this is good to go. Can someone kindly review? |
| DEPENDS_ON RAJA umpire cuda) | ||
|
|
||
| blt_add_executable( | ||
| NAME 10_raja_launch_transpose_solution |
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.
Shall I change the directory structure in the Views lesson too?
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.
oh if you want, we can do it in one go after we merge the lessons in
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.
Yes, let's move the solution files up one directory level and build them with the rest of the code after all other content is in. I've got an issue where we can collect changes like this to address #45
|
LGTM One note for the future: Maybe we could set one of the views as |
yeah that could be cool. |
No description provided.