microsoft/SynapseML

UDFtransformer should not extend HasInputCol and HasInputCols

Open

#243 geöffnet am 13. Feb. 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)Scala (5.228 Stars) (861 Forks)batch import
buggood first issuehelp wanted

Beschreibung

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

Tech Stack
scala
Domain
machine learning
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
ScalaSpark MLSynapseML
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
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.