-
Notifications
You must be signed in to change notification settings - Fork 285
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
Init cudagraph #424
base: main
Are you sure you want to change the base?
Init cudagraph #424
Conversation
Thank you so much for restarting this thread ! Can you add some basic tests similar to https://gitlab.tiker.net/kaushikcfd/pycuda/-/merge_requests/2/diffs#653379dd0e00d91ae08b155f35494fd427d5c80e |
Co-authored-by: Mit Kotak <[email protected]> Co-authored-by: Gerlof Fokkema <[email protected]>
@nimlgen Getting a segfault for
|
Thanks for testing. Should be fixed now (would appreciate if you check on your end as well). LMK if any other refactors are needed. |
Hey @inducer, anything I can do to help get this upstreamed? |
Thanks for jumping in ! Here's a non-exhaustive list of node calls that I am working on porting from my PR to this one:
Along with the Feel free to pick your favorite node :) Hopefully that helps. |
This is one more implementation of cuda graphs baed on 2 previous MR. This is a first MR, so has some basic functionality to create CUDAgraphs + setting some dynamic parameters to nodes.
I'd love your comments and feedback.