Skip to content

Conversation

johnnynunez
Copy link

@johnnynunez johnnynunez commented Sep 2, 2025

Closes: #485 @inducer

Copy link

@lahwaacz lahwaacz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me, thanks!

@mikeseven
Copy link

me too. awesome!

@johnnynunez
Copy link
Author

@inducer can you merge it?

@inducer
Copy link
Owner

inducer commented Sep 9, 2025

Sorry, I won't be merging this since retaining the primary context isn't equivalent to creating one. See #487 for a nominally equivalent fix.

Note that I don't disagree that retaining the primary context is likely better in many cases than creating a fresh one, but there already exists functionality for that.

@inducer
Copy link
Owner

inducer commented Sep 9, 2025

Nonetheless thanks for working on this!

@inducer inducer closed this Sep 9, 2025
@johnnynunez
Copy link
Author

@inducer thank u! I was only trying to help💚

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.

pycuda doesn't build with cuda 13
4 participants