alteryx / alteryx/evalml

Make `_drop_time_index()` public and call when using `transform_all_but_final()`

Open
#4,028 0 comments 0 reactions 0 assignees View on GitHub
enhancement refactor
Dominant language
Python
Stars
850
Forks
96
PR merge metrics
No merged PRs in 30d

Description

- As a user of EvalML, I expect an estimator-ready dataframe when I call `transform_all_but_final()`. However, for time series problems, this dataframe includes the datetime column even if it is dropped before calling the estimator. Thus, the time index column should be dropped when `transform_all_but_final()` is called.
- In addition, as a user of EvalML, I would like to have a built-in pipeline function that I can call that drops the time index column. This exists as `_drop_time_index()`, but is not meant to be called publicly.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.