Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOCS] Apply WindowStatisticsTransform to regressors #473

Closed
egoriyaa opened this issue Sep 9, 2024 · 0 comments · Fixed by #474
Closed

[DOCS] Apply WindowStatisticsTransform to regressors #473

egoriyaa opened this issue Sep 9, 2024 · 0 comments · Fixed by #474
Labels
documentation Improvements or additions to documentation

Comments

@egoriyaa
Copy link
Collaborator

egoriyaa commented Sep 9, 2024

📖 Documentation improvement

We need to clarify that transforms inherited from WindowStatisticsTransform in most cases should be applied to regressor columns.

For example "This transform, applied to non-regressor column, generates non-regressor column. Apply it to regressor columns to get regressor columns too. In majority of cases you need to generate regressor to use them in the future. For example, apply this transform to target lags, not to target directly."

May be add warning in constructor if transform is applied to non-regressor columns.

Additional context

No response

@egoriyaa egoriyaa added the documentation Improvements or additions to documentation label Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant