At the moment things like OrdinalPatterns accept m as a keyword. This "parameter" m is then cast into a type parameter. However, because m corresponds to the length of a static vector, it is better to provide it explicitly as a type paraemter, rather than deciding the type by the value of m.
So, this means, change
OrdinalPattern(; m, kwargs...)
to
OrdinalPattern{m}(; kwargs...)
this ensures that the type m of OrdinalPattern can be statically deduced and propagated without a type instability.
The method with m as keyword needs to be deprecated and throw a deprecation warning.
This change will increase performance and compilation speed.
At the moment things like
OrdinalPatternsacceptmas a keyword. This "parameter"mis then cast into a type parameter. However, becausemcorresponds to the length of a static vector, it is better to provide it explicitly as a type paraemter, rather than deciding the type by the value ofm.So, this means, change
to
this ensures that the type
mofOrdinalPatterncan be statically deduced and propagated without a type instability.The method with
mas keyword needs to be deprecated and throw a deprecation warning.This change will increase performance and compilation speed.