Skip to content

makes computeRadiusPart consider all the spheres radius in the domain

Benjamin Dedieu requested to merge fixHydroForceEngineRadiusPart into master

This commit fixes a bug related to the fact that the spheres considered by averageProfile() are not the same as those considered by computeRadiusPart(). Specifically, computeRadiusPart() uses particle IDs passed to HydroForceEngine by the user, whereas averageProfile() considers all spheres in the domain. This can lead to bugs when HydroForceEngine is given only the IDs of dynamic particles, excluding IDs of other particles, such as those fixed at the bottom of the domain. In such cases, if the radius of fixed particles differs from those passed to HydroForceEngine, it results in the error 'Radius of particle not found.'

The method used in this commit is to consider all spheres in the domain instead of just IDs when computing the radiusParts vector.

Merge request reports