Evalml: Aktualisieren Sie das Flag "raise_errors" für die automatische ml-Suche auf den Standardwert "true".

In #252 hat @angela97lin einen Fehler behoben, der unentdeckt blieb, weil unsere Unit-Test-Abdeckung von automl fit so konfiguriert war, dass keine Fehler ausgelöst wurden (insbesondere ist der Parameter raise_errors derzeit standardmäßig auf false eingestellt). Ihr PR hat die Unit-Tests aktualisiert, um raise_errors bei jedem Aufruf von automl fit in den Tests auf true zu setzen.

Ich wollte ein Ticket erstellen, um dies etwas weiter zu besprechen. Ich bin verwirrt über den Parameter raise_errors . Warum existiert es? Warum gilt es nur für automl fit ? Wenn es ein notwendiger Parameter ist, gibt es ein besseres Design, und insbesondere eines, das Fehler wie diesen abschrecken würde? Ich mache mir Sorgen, dass wir wieder auf ähnliche Probleme stoßen könnten, wenn wir uns darauf verlassen, dass das Team daran denkt, raise_errors in den Komponententests auf true zu setzen.

Ideen: Ich weiß, dass @kmax12 dafür eine Umgebungsvariable verwendet hat . Wenn dieser Parameter erforderlich ist, aber meine Bedenken hinsichtlich des Unit-Tests gültig sind, könnte dies eine gute Lösung sein. Wir könnten auch alle Tests aktualisieren, um ein Testgerät zu verwenden, das irgendwie sicherstellt, dass raise_errors richtig eingestellt ist, entweder über env var, eine andere globale Konfiguration oder durch Umschließen der automl-Methode fit .