Skip to content

[com1] add checking for zero mass of particle and remove properly#1180

Merged
fso42 merged 3 commits intomasterfrom
rmovePart
Sep 10, 2025
Merged

[com1] add checking for zero mass of particle and remove properly#1180
fso42 merged 3 commits intomasterfrom
rmovePart

Conversation

@awirb
Copy link
Copy Markdown
Contributor

@awirb awirb commented Sep 8, 2025

if mass of a particle becomes 0 (e.g. because of detrainment) previously particle was removed after dividing by m=0 and resulting inf or nan velocities - that lead to new particle positions that caused iCellNew < 0
now first check if m=0, then continue with next particle and remove this one

TODO - update docu-> @PaulaSp3

@awirb awirb requested review from PaulaSp3 and fso42 September 8, 2025 09:23
@fso42 fso42 added the enhancement New feature or request label Sep 8, 2025
@fso42 fso42 added this to the Version 1.14 milestone Sep 8, 2025
@fso42 fso42 changed the title add checking for zero mass of particle and remove properly [com1] add checking for zero mass of particle and remove properly Sep 8, 2025
@awirb awirb assigned fso42 and unassigned awirb Sep 10, 2025
awirb and others added 3 commits September 10, 2025 17:06
Co-authored-by: Paula Spannring <95042192+PaulaSp3@users.noreply.github.com>
stopped particles are always and only deleted and stored in stoppedParticles when surface is adapted due to stopping

little comment

Update avaframe/com1DFA/DFAfunctionsCython.pyx

Co-authored-by: Anna Wirbel <68687423+awirb@users.noreply.github.com>
@fso42
Copy link
Copy Markdown
Contributor

fso42 commented Sep 10, 2025

Standardtests ident

@fso42 fso42 merged commit a17449e into master Sep 10, 2025
5 checks passed
@fso42 fso42 deleted the rmovePart branch September 10, 2025 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants