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
In principle this sounds like a good idea. However, we do have UnivariateTimeTypeToContinuous and I would prefer to avoid code duplication.
Perhaps we could add date_time_encoder=UnivariateTimeTypeToContinous() as a new ContinuousEncoder hyperparameter, and use that to treat columns with scitype ScientificTimeType (corresponding to any machine type <: Dates.TimeType).
Right now, inputting a date into a
ContinuousEncoder
just drops it, but it seems more reasonable to handle by converting dates into a number of days.The text was updated successfully, but these errors were encountered: