Skip to content

[WIP] Add Inputer as DOperation#68

Open
jaroslaw-osmanski wants to merge 3 commits intospark-2.2from
add-imputer
Open

[WIP] Add Inputer as DOperation#68
jaroslaw-osmanski wants to merge 3 commits intospark-2.2from
add-imputer

Conversation

@jaroslaw-osmanski
Copy link

No description provided.

}


class Imputer extends TransformerAsOperation[InputerTransformer] with OperationDocumentation {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see Imputer and ImputerModel in Spark, which means Imputer is an Estimator which trains ImputerModel, which is in turn used to transform dataframes. We should have it reflected in seahorse. You can do it in a manner similar to how we handle it in other operations, using SparkEstimatorWrapper, SparkModelWrapper and than use EstimatorAsOperation on wrapped estimator.

@jaroslaw-osmanski jaroslaw-osmanski force-pushed the spark-2.2 branch 2 times, most recently from 71203a1 to ed4adca Compare May 18, 2018 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants