-
-
Notifications
You must be signed in to change notification settings - Fork 38
Comparing changes
Open a pull request
base repository: SciML/JumpProcesses.jl
base: v9.22.0
head repository: SciML/JumpProcesses.jl
compare: v9.22.1
- 6 commits
- 6 files changed
- 2 contributors
Commits on Feb 23, 2026
-
fix data race in resetted_jump_problem, decorrelate jump/noise seeds
Remove dead randexp!(_jump_prob.rng, ...) blocks from resetted_jump_problem and reset_jump_problem! that caused a data race when called from multiple threads via EnsembleThreads. The jump_u values they wrote were unconditionally overwritten by VR_FRMEventCallback's initialize method. Add _derive_jump_seed to decorrelate the jump aggregator's seed from the noise process seed when StochasticDiffEq passes the same seed to both. Add test/ensemble_problems.jl covering serial stream uniqueness, sequential solve independence, threaded completion (no data race), seed-based stream independence, and VR_FRM jump_u threshold uniqueness. Co-Authored-By: Claude Opus 4.5 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 36b76ec - Browse repository at this point
Copy the full SHA 36b76ecView commit details -
use hash + Xoshiro for seed derivation, add salt constant
Strengthens _derive_jump_seed by hashing the input seed with a fixed salt before passing through Xoshiro. This ensures decorrelation from the caller's seed regardless of the JumpProblem's stored RNG type. Co-Authored-By: Claude Opus 4.5 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 899e27e - Browse repository at this point
Copy the full SHA 899e27eView commit details -
add first jump time uniqueness checks to ODE+VR ensemble tests
Co-Authored-By: Claude Opus 4.5 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b710e80 - Browse repository at this point
Copy the full SHA b710e80View commit details -
fix CI: update jump_u test to use integrator, remove stale randexp! i…
…mport The variable_rate.jl test was checking prob.u0.jump_u, but VR_FRM's callback initialize sets integrator.u.jump_u (a copy). Updated the test to use init() and check the integrator's jump_u values directly. Also removed the stale randexp! import that was flagged by ExplicitImports. Co-Authored-By: Claude Opus 4.5 <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 4903df6 - Browse repository at this point
Copy the full SHA 4903df6View commit details -
Merge pull request #556 from isaacsas/fix_data_race_condit
fix data race in resetted_jump_problem, decorrelate jump/noise seeds
Configuration menu - View commit details
-
Copy full SHA for 2d1f9d7 - Browse repository at this point
Copy the full SHA 2d1f9d7View commit details -
Configuration menu - View commit details
-
Copy full SHA for ca5d2ed - Browse repository at this point
Copy the full SHA ca5d2edView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v9.22.0...v9.22.1