Evalml: Atualizar o sinalizador de pesquisa automática "raise_errors" para o padrão para verdadeiro

Em # 252, @ angela97lin corrigiu um bug que não foi descoberto porque nossa cobertura de teste de unidade de automl fit foi configurada para não gerar erros (especificamente, o parâmetro raise_errors atualmente é falso). Seu PR atualizou os testes de unidade para definir raise_errors como verdadeiro em cada chamada para automl fit nos testes.

Eu queria criar um tíquete para discutir isso um pouco mais. Estou confuso sobre o parâmetro raise_errors . Por que existe? Por que ele se aplica apenas ao automl fit ? Se for um parâmetro necessário, existe um design melhor e, em particular, um que desencorajaria bugs como este? Estou preocupado porque, se confiarmos na equipe para lembrar de definir raise_errors como verdadeiro nos testes de unidade, poderemos ter problemas semelhantes novamente.

Idéias: Eu sei que @ kmax12 mencionou o uso de uma variável de ambiente para isso . Se esse parâmetro for necessário, mas minhas preocupações com o teste de unidade forem válidas, essa pode ser uma boa solução. Também poderíamos atualizar todos os testes para usar uma fixação de teste que de alguma forma garante que raise_errors seja definido apropriadamente, seja via env var, outra configuração global, ou envolvendo o método automl fit .