microsoft/SynapseML

UDFtransformer should not extend HasInputCol and HasInputCols

Closed

#243 opened on Feb 13, 2018

View on GitHubContributor guide
 (1 comment) (0 reactions) (1 assignee)Scala (861 forks)batch import
buggood first issuehelp wanted

Repository metrics

Stars
 (5,228 stars)
PR merge metrics
 (PR metrics pending)

Description

This is a bit confusing and will cause complications if both parameters are independently set. Try extending the latter and adding custom setters in scala and py side

Contributor guide

Research direction
Understand the current inheritance hierarchy of UDFtransformer and implement custom setters in Scala and Python to avoid confusion between HasInputCol and HasInputCols.
Tech stack
scalapython
Domain
machine learningdata
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitScalaPython
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70