Summary from feedback round

[zzril]

Make proper issues out of these:

• OneHotEncoder: Doku etwas "abstrakt" ( -> Codebeispiele?)
• OneHotEncoder: schwierig zu benutzen (Doku?), Beispiel wäre sehr gut (Referenz auf "Dummies", wie sie bei pandas heißen, wäre auch praktisch)
  • Improve documentation of OneHotEncoder #181
• OneHotEncoder bzw. alle Transformer: Doku sollte klarstellen, dass Spaltennamen sich ändern; Zugriff auf neue Spaltennamen (oder gelöschte) wäre hilfreich
  • Columns that are added/changed/removed by a TableTransformer  #190
• "Wann werden Kopien zurückgegeben, wann wird in-place gearbeitet?"
  • Always specify explicitly that methods do not change an object in place but return a new one #189
• OneHotEncoder: Unklar, dass er überhaupt zu benutzen ist (siehe unten); allgemein: Interface mit fit, predict, transform etc. besser klarmachen
  • Improve error handling of classifiers and regressors #153 & Improve documentation of OneHotEncoder #181
• mehrfache Verwendung von fit (bei Klassifizierer und Transformer) evtl. ungünstig -> Discussion?
  • Würde ich für den Moment nicht ändern, da das fit schon die gleiche Bedeutung bei beidem hat (@lars-reimann)
• (evtl. mehr Querverweise in der Doku)
  • Darauf sollten wir auf jeden Fall achten. Ich finde es allerdings schwierig, das als Issue festzuhalten. Wir sollten eher unsere Development Guidelines erweitern.
  • Explain in the development guidelines how to write good docstrings #191
• (Zur Poetry-Doku hinzufügen: poetry add safe-ds)
  • Für den Moment würde ich das nicht ändern. Die Personen, die poetry nutzen, wissen bereits wie sie damit neue Abhängigkeiten hinzufügen können. Der Verweis zu pip ist für die absoluten Einsteiger, die gerade Python installiert haben (@lars-reimann)
• Scikit-Fehlermeldungen abfangen (und durch eigene ersetzen) (z.B., wenn nicht-numerische Daten benutzt werden -> dann in der neuen Exception direkt vorschlagen, was zu machen ist)
  • Improve error handling of classifiers and regressors #153
• Doku / Codebeispiel zum OneHotEncoder sagt verwirrenderweise TableTransformer (Inkonsistenz?, evtl. darauf hinweisen?)
  • For now I would favor Remove docstrings from overriding methods #72 to reduce duplication of docstrings and make them easier to keep updated. Once Remove docstrings from overriding methods #72 is done, we can no longer change TableTransformer to OneHotEncoder (@lars-reimann )

[lars-reimann]

I've created individual issues and linked them in the opening post.