Closed spaette closed 8 months ago
@Jay-sanjay
My preference would be to mark the pull a Draft until the matters of this comment are resolved. Presumably you know how to do a soft reset if you make changes to the pull request. It would be simpler to include any such modifications in a single commit and then do a soft reset. I have found no example use of the strings Continuating
/continuate
in any location other than this software repository.
Incidentally, if you replace "fixing issue" in your pull comment with "Closes" this ticket will automatically close when the pull is merged.
the patch has penalized
which is incorrect rather than penalize
on this line
src/tipping/mfs.jl
if mapper(u0 + perturbation) == mapper(u0)
- # penaltize if we stay in the same basin:
+ # penalized if we stay in the same basin:
return dist + penalty
The use of
Continuating
/continuate
is unusual.
these were not part of this ticket
CHANGELOG.md
- New function `continuation` that calculates basins fractions and how these change versus a parameter (given a continuation method)
- - New basins fraction continuation method `RecurrencesFindAndMatch` that utilizes a brand new algorithm to continuate basins fractions of arbitrary systems.
+ - New basins fraction continuate method `RecurrencesFindAndMatch` that utilizes a brand new algorithm to continuate basins fractions of arbitrary systems.
- `match_attractor_ids!` has been fully overhauled to be more flexible, allow more ways to match, and also allow arbitrary user-defined ways to match.
src/continuation/continuation_recurrences.jl
progress = ProgressMeter.Progress(length(prange);
- desc="Continuating basins fractions:", enabled=show_progress
+ desc="continuate basins fractions:", enabled=show_progress
)
possibly reappears
/reappear
could be used here
$ grep -nr re\-appear Attractors.jl
Attractors.jl/src/continuation/match_attractor_ids.jl:224: # and re-appears, it will get a different (incremented) id as it should!
Attractors.jl/docs/src/index.md:15:- New function [`match_continuation!`](@ref) which improves the matching during a continuation process where attractors dissapear and re-appear.
$
Convenience Minimal Silhouettes Unknown accumulated approximation arbitrarily assigns attractor construct converged choose deterministic discretization dimensional disappear disappears explicitly function guaranteed halt halting implemented initializes internal just original originally output penalize penalized perturbation perturbations silhouette statistics system system threshold unnecessary
The use of
Continuating
/continuate
is unusual.