removing rnd reuse logic (disabled by default)

[emmacware]

No description provided.

[slayoo]

OK, in the 2D kinematic setup, the reuse logic is enabled by default:

PySDM/examples/PySDM_examples/Morrison_and_Grabowski_2007/common.py

Line 32 in 20c8340

self.coalescence_optimized_random = True

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.46%. Comparing base (20c8340) to head (94ce24c).
⚠️ Report is 30 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1642      +/-   ##
==========================================
- Coverage   85.53%   85.46%   -0.08%     
==========================================
  Files         396      394       -2     
  Lines        9601     9541      -60     
==========================================
- Hits         8212     8154      -58     
+ Misses       1389     1387       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[slayoo]

IIUC, with the reuse disabled, we are shuffling all n_sd numbers regardless of how many are actually needed in a given substep. So, I'd suggest that before removing the reuse logic, we implement a scheme to alter the amount of shuffled numbers depending on how many are actually needed in a given substep

[github-actions]

Stale pull request message