alteryx / alteryx/evalml

Either fix warnings, or suppress only inside evalml code rather than globally

Aperta
#1,368 0 commenti 0 reazioni 1 assegnatario Rivendicata da @asniyaz Vedi su GitHub
enhancement
Lingua principale
Python
Stelle
850
Fork
96
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

#### Background
We have [some code in our main `__init__.py`](https://github.com/alteryx/evalml/blob/main/evalml/__init__.py#L16) which suppresses warnings from `skopt`, as well as `FutureWarning`/`DeprecationWarning` and [one warning we started getting recently from pandas.](https://github.com/alteryx/evalml/blob/main/evalml/__init__.py#L22)

#### Problem
As a user of evalml, I want to use evalml in my software application, without evalml changing any global state like what warnings do or don't appear in the system. Maybe I want to see `FutureWarning`/`DeprecationWarning` or that pandas warning for my codebase.

#### Proposal
Let's try removing these warning suppressions and see what happens. Its possible later versions have cleared up the `skopt` warnings. We may want to take action based on some of the `FutureWarning`/`DeprecationWarning` issues which come up.

As for the pandas warning, that feels ok to leave in place. The question I have there is, where in our codebase is that warning being emitted? Can we change the way we use pandas instead to avoid that warning?

@christopherbunn @thehomebrewnerd FYI this is RE our conversation on #1358

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.