Skip to content

Adapt to CudaRdcUtils #376

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

Closed
wants to merge 3 commits into from
Closed

Adapt to CudaRdcUtils #376

wants to merge 3 commits into from

Conversation

pcanal
Copy link

@pcanal pcanal commented Apr 1, 2025

This is an extension of #372

@phsft-bot
Copy link

Can one of the admins verify this patch?

@agheata
Copy link
Contributor

agheata commented Apr 1, 2025

Thanks @pcanal ! I did practically the same but did not work for me. I'll try it tomorrow.

@pcanal
Copy link
Author

pcanal commented Apr 1, 2025

Thanks. All those are necessary and the derived code also needs to use the cuda_rdc_ (or be willing to jungle with sharp knifes and hope for the best :) ). Any errors would indicates that we either missed a spot or (hopefully not) that there is still issue to be solved in CudaRdcUtils

@agheata agheata mentioned this pull request Apr 16, 2025
agheata added a commit that referenced this pull request Jun 3, 2025
This overseeds #376 rebasing and
adding to the changes in the master

---------

Main author: Philippe Canal <[email protected]>
@agheata
Copy link
Contributor

agheata commented Jun 3, 2025

We merged in the master the rebased version of this. Thanks @pcanal !

@agheata agheata closed this Jun 3, 2025
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.

3 participants