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

Update the pmacc tuple and its use in the binning plugin #5264

Draft
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

ikbuibui
Copy link
Contributor

@ikbuibui ikbuibui commented Feb 3, 2025

Add a tuple size implementation to the pmacc tuple
Use pmacc tuple on the host side as well
Add a new function applyEnumerated which passes pairs of (index ,tuple) to the callable argument
Fix forwarding of arguments in binning::apply and tupleMap

@ikbuibui ikbuibui added the component: plugin in PIConGPU plugin label Feb 3, 2025
@ikbuibui ikbuibui added this to the 0.9.0 / next stable milestone Feb 3, 2025
@psychocoderHPC psychocoderHPC added the bug a bug in the project's code label Feb 4, 2025
psychocoderHPC
psychocoderHPC previously approved these changes Feb 4, 2025
@psychocoderHPC
Copy link
Member

@ikbuibui please rebase agains tthe current dev branch

psychocoderHPC
psychocoderHPC previously approved these changes Feb 4, 2025
@psychocoderHPC
Copy link
Member

This pr should be rebased/merged after #5266 is in the dev branch to ensure we not break anything. Currently the CI is only running g++-CUDA tests.

@ikbuibui ikbuibui marked this pull request as draft February 6, 2025 12:57
@ikbuibui ikbuibui force-pushed the tuple_update branch 5 times, most recently from 30e7a84 to e91d763 Compare February 7, 2025 10:34
@ikbuibui ikbuibui force-pushed the tuple_update branch 2 times, most recently from 170f885 to 01d0e62 Compare February 7, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug a bug in the project's code component: plugin in PIConGPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants