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
Currently, the isless_rand function, which we use as the default value comparator for OrdinalPatternEncoding, will not give reproducible results. This is because we don't provide an rng argument to the rand call.
To solve this, we could either:
Force the user to provide their custom lt function, taking care of reproducibility themselves (not preferable IMHO)
Include rng as a field to both OrdinalPatternEncoding and OrdinalPatterns, which gets passed on to isless_rand.
Or maybe there's a better way?
The text was updated successfully, but these errors were encountered:
Note that this is only a problem if there are duplicate datapoints in the timeseries.
The problem occurs if there are tied values inside any state vector, since it is the individual state vectors that are sorted to map them onto an ordinal pattern.
Currently, the
isless_rand
function, which we use as the default value comparator forOrdinalPatternEncoding
, will not give reproducible results. This is because we don't provide anrng
argument to therand
call.To solve this, we could either:
lt
function, taking care of reproducibility themselves (not preferable IMHO)rng
as a field to bothOrdinalPatternEncoding
andOrdinalPatterns
, which gets passed on toisless_rand
.Or maybe there's a better way?
The text was updated successfully, but these errors were encountered: