Skip to content

Transform knows if it converts data to regressor #326

Closed
1 task done
julia-shenshina opened this issue Nov 29, 2021 · 0 comments · Fixed by #361
Closed
1 task done

Transform knows if it converts data to regressor #326

julia-shenshina opened this issue Nov 29, 2021 · 0 comments · Fixed by #361
Assignees
Labels
enhancement New feature or request

Comments

@julia-shenshina
Copy link
Contributor

julia-shenshina commented Nov 29, 2021

🚀 Feature Request

Now each transform knows that it can transform the column to regressor but does not contain the info directly. In this issue we need to change it.

Motivation

New way of work with regressors.

Proposal

Create empty mixin FutureMixin

For each transform

  • add FutureMixin to the list of ancestors, if the transform can convert non-regressor column to a regressor one
  • change work with output_column:
    • if output_column is None, set output_column as transform.__repr__ without any prefixes

Test cases

Update tests for output_column names for all the transforms

Alternatives

No response

Additional context

No response

Checklist

  • I discussed this issue with ETNA Team
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants