microsoft/SynapseML

UDFtransformer should not extend HasInputCol and HasInputCols

Open

#243 ouverte le 13 févr. 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)Scala (5 228 stars) (861 forks)batch import
buggood first issuehelp wanted

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

Guide contributeur

Stack technique
scala
Domaine
machine learning
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
ScalaSpark MLSynapseML
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
The issue suggests that UDFTransformer currently extends both HasInputCol and HasInputCols, which can cause confusion if both parameters are set independently. The fix should extend only HasInputCols and add custom setters for inputCol and inputCols on the Scala side, with corresponding updates on the Python side. Look at the current UDFTransformer source code in the SynapseML repository to understand the inheritance chain and implement the change.