Add number of split integration time steps to summary#1194
Conversation
There was a problem hiding this comment.
Pull request overview
The PR improves the simulation finish summary printed by InfoCallback, switching to a vertically aligned multi-line format and adding a new line that reports accepted/total time steps from the inner split-integration integrator when present.
Changes:
- Reformat the "simulation finished" summary using
@printfwith column alignment. - Add a new "Split integration time steps:" line when
integrator.p.split_integration_datais notnothing. - Extend the info callback tests to cover both the standard and split-integration cases.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/callbacks/info.jl | Rewrites print_summary with @printf formatting and conditionally prints split-integration step counts. |
| test/callbacks/info.jl | Updates the existing finished-summary test for the new format and adds a new testset for the split-integration case. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1194 +/- ##
==========================================
- Coverage 89.88% 89.87% -0.02%
==========================================
Files 133 133
Lines 10452 10459 +7
==========================================
+ Hits 9395 9400 +5
- Misses 1057 1059 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Before
With split integration:
Now
With split integration: