Skip to content

Avoid DDMC cell update prior to implicit capture absorption (bug). - #37

Merged
RyanWollaeger merged 2 commits into
developfrom
fix_impcapt_ddmc
Aug 10, 2026
Merged

Avoid DDMC cell update prior to implicit capture absorption (bug).#37
RyanWollaeger merged 2 commits into
developfrom
fix_impcapt_ddmc

Conversation

@RyanWollaeger

Copy link
Copy Markdown
Collaborator

Background

  • Implicit capture requires cell-local deposition, but the DDMC algorithm assumed that leakage always meant the cell could be immediately updated.

Description of Changes

  • Remove i,j,k updates in ddmc_ptcl_step

Checklist

  • New features are documented
  • Tests added for bug fixes and new features
  • (@lanl.gov employees) Update copyright on changed files

@RyanWollaeger

Copy link
Copy Markdown
Collaborator Author

This needs a bit more work - its not compatible with MG at the moment.

@RyanWollaeger
RyanWollaeger force-pushed the fix_impcapt_ddmc branch 2 times, most recently from 6ac15ef to fd15986 Compare August 8, 2026 00:08
@RyanWollaeger

Copy link
Copy Markdown
Collaborator Author

This needs a bit more work - its not compatible with MG at the moment.

This nominally should be ready with the introduction of _next indices in the DDMC logic preceding the absorption

+ Add [ip,jp,kp]_next to use in ptcl_ddmc_step, to avoid direct particle index update.
@RyanWollaeger
RyanWollaeger merged commit 3f60f88 into develop Aug 10, 2026
3 checks passed
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.

1 participant