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

Add example notebooks for compiling with opencv and an adaptation of a CUDA training series #34

Merged
merged 2 commits into from
Jun 3, 2024

Conversation

cosminc98
Copy link
Collaborator

@cosminc98 cosminc98 commented May 19, 2024

I am trying to adapt this CUDA training series to use our library in order to create possibly the most efficient way to learn CUDA C++ available on the internet. The first session was adapted, but I would require some feedback. @andreinechaev If you have the time, you can run the notebook on either Colab or Kaggle. I will add a README.md in the "notebooks" directory at some point describing how to import from Kaggle these notebooks straight from GitHub.

@andreinechaev
Copy link
Owner

I am trying to adapt this CUDA training series to use our library in order to create possibly the most efficient way to learn CUDA C++ available on the internet. The first session was adapted, but I would require some feedback. @andreinechaev If you have the time, you can run the notebook on either Colab or Kaggle. I will add a README.md in the "notebooks" directory at some point describing how to import from Kaggle these notebooks straight from GitHub.

Great, nice initiative @cosminc98 I will check it when I have time. Will try to do it ASAP

@andreinechaev
Copy link
Owner

@cosminc98 Thank you for your contribution. I think you provided clean notebooks, it is easy to read and to follow them.
If you think that we should merge, I can proceed.
If you're looking for enhancements, CUDA is very verbose, so splitting the solution in multiple cells, can make it even better. It will involve splitting the files, so it might be an overkill. Anyway, I leave it for your judgment.
Thanks!

@cosminc98
Copy link
Collaborator Author

@andreinechaev Thanks for the feedback! I'll try to have a look ASAP if there's a way to split them in a way to be as easy to read as possible.

@cosminc98
Copy link
Collaborator Author

@andreinechaev I've split the code in the OpenCV notebook into more manageable pieces. As for the CUDA intro notebook, I think it may be a bit too confusing to beginners to have the kernel function in a separate cell (as I think that would be the only split that can be reasonably done). If it's all in one cell, they can immediately run the cell and see if they got the correct assignment solution. I think we can introduce more modularity in the next notebooks, as users will have had a bit more experience (with both CUDA and the library). What do you think?

@andreinechaev
Copy link
Owner

@andreinechaev I've split the code in the OpenCV notebook into more manageable pieces. As for the CUDA intro notebook, I think it may be a bit too confusing to beginners to have the kernel function in a separate cell (as I think that would be the only split that can be reasonably done). If it's all in one cell, they can immediately run the cell and see if they got the correct assignment solution. I think we can introduce more modularity in the next notebooks, as users will have had a bit more experience (with both CUDA and the library). What do you think?

I see, thank you @cosminc98 . I believe it's a reasonable explanation. The notebook looks good. Shall we proceed with merging?

@cosminc98 cosminc98 merged commit 0b60c36 into master Jun 3, 2024
7 checks passed
@andreinechaev andreinechaev deleted the feature/first-example-notebooks branch June 4, 2024 08:42
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

Successfully merging this pull request may close these issues.

None yet

2 participants