Skip to content

Add aggregation method parameter to Imputers #1015

Open
alex-hse-repository opened this issue Nov 25, 2022 · 0 comments
Open

Add aggregation method parameter to Imputers #1015

alex-hse-repository opened this issue Nov 25, 2022 · 0 comments
Labels
DKO enhancement New feature or request

Comments

@alex-hse-repository
Copy link
Collaborator

🚀 Feature Request

Add possibility to perform different kids of aggregation for seasonal strategy in TimeSeriesImputerTransform

Proposal

  1. In TimeSeriesImputerTransform add parameter aggregation_method: AggregationMode = "AggregationMode.mean -- method to aggregate values in the window (see link)
  2. Update method _fill to handle the logic with aggregation_method -- now it should aggregate values according the the passed method
    df.iloc[i] = np.nanmean(df.iloc[indexes])
  3. Update the docs

Test cases

Add test for the seasonal strategy which checks that it works correctly with different aggregation_method

Additional context

No response

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DKO enhancement New feature or request
Projects
Status: Specification
Development

No branches or pull requests

2 participants