Fix convergence time for recurrences (#171)

Now it cannot be less than Δt, and it is also type stable.

Closes #127

Author: Datseris

Project.toml

@@ -2,7 +2,7 @@ name = "Attractors"
 uuid = "f3fd9213-ca85-4dba-9dfd-7fc91308fec7"
 authors = ["George Datseris <[email protected]>", "Kalel Rossi", "Alexandre Wagemakers"]
 repo = "https://github.com/JuliaDynamics/Attractors.jl.git"
-version = "1.24.3"
+version = "1.24.4"
 
 
 [deps]

src/mapping/recurrences/attractor_mapping_recurrences.jl

@@ -180,10 +180,11 @@ function convergence_time(m::AttractorsViaRecurrences)
         # time of neighboring grid cells.
         x = get(kw, :consecutive_recurrences, 100) + get(kw, :attractor_locate_steps, 1000)
     elseif m.bsn_nfo.return_code == :bas_hit
-        return NaN
+        return typemax(bsn_nfo.Δt)
     else
         x = get(kw, :consecutive_attractor_steps, 2)
     end
+    x = max(i, x) # it cannot be more than i!
     return (i - x + 1)*m.bsn_nfo.Δt
 end