You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
These tests involve remake of NonlinearProblems with remove_conserved = true. The conservation law parameter Gamma and eliminated species values don't update correctly.
events.jl:336-339, 363-365 - Solution parameter access after events (MTK #4030) - NOW FIXED
dsl_options.jl:663 - Integer symtype inference - test removed/was never broken
simulate_jumps.jl:249,265 - Integer type preservation for jumps (MTK #4170) - FIXED by c314ed5f7
events.jl:145,236 - ReactionSystem equality with events - FIXED by switching to Catalyst.isequivalent
dsl_options.jl:1609,1629 - isequal for systems with events - FIXED by switching to Catalyst.isequivalent
custom_crn_functions.jl:201-203 - Events equality after deepcopy - FIXED by switching to Catalyst.isequivalent/continuous_events_equal/discrete_events_equal
reactionsystem_serialisation.jl:358,360 - Serialised system equality with events - FIXED by switching to Catalyst.isequivalent
custom_crn_functions.jl:218-219 - Expanded event affects were nothing - FIXED by correcting expand_registered_functions to properly pass affect eqs, handle affect_neg, and preserve discrete_parameters
Removed/Commented Out Tests
The following tests that were previously @test_broken have been fully commented out:
latexify.jl:178 - Changed to comment, not @test_broken
latexify.jl:279, 282 - Entire test block commented out (extend/compose issues with System+ReactionSystem)
Recommendations
Medium Priority
Investigate conservation law remake issues for NonlinearProblems