Closed Datseris closed 1 year ago
Merging #64 (c853b47) into main (9453971) will increase coverage by
0.29%
. The diff coverage is94.11%
.
@@ Coverage Diff @@
## main #64 +/- ##
==========================================
+ Coverage 70.22% 70.51% +0.29%
==========================================
Files 20 20
Lines 1105 1116 +11
==========================================
+ Hits 776 787 +11
Misses 329 329
Impacted Files | Coverage Δ | |
---|---|---|
src/continuation/aggregate_attractor_fractions.jl | 94.73% <ø> (ø) |
|
src/deprecated.jl | 0.00% <ø> (ø) |
|
src/continuation/continuation_recurrences.jl | 88.88% <75.00%> (ø) |
|
src/continuation/continuation_grouping.jl | 97.72% <100.00%> (ø) |
|
src/continuation/match_attractor_ids.jl | 72.22% <100.00%> (+5.00%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
All seems good to me, just the two small typos :)
The new names are catchy. I don't see any problem since they are used in the context of continuation.
Good for me
rematch!
to do different matching after the continuation has stoppedRecurrencesFindAndMatch
andGroupAcrossParameter
. I've droppedContinuation
from both names, as they subtype theAttractorsBasinsContinuation
abstract typeLet me know if you disagree.