-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
warnings.filterwarnings -- please don't do this. it's bad behavior. #11792
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Could you please elaborate in what way this is a problem for your application? From the comments, it sound like it should only affect (Also see related discussion in #6930) |
Could you please elaborate on why If you add The problem is that my applications global state should not be affected by importing your package. I also wouldn't want you doing things like changing my logging configuration. |
Tried, that, by installing sklearn from sources, and manually raising a deprecation in I agree the current situation is bad, I was just curious to know what kind of practical issues you run into with this setup (beyond the fact that it's bad on principle). Having a good understanding of the use cases, would also help us to come up with the right solution. This problem was discussed in #9857 (see in particular #9857 (comment) and #9857 (comment)) which outlines the reason for the current situations. Closing this as a duplicate of #9857 (to avoid fragmenting the same discussion over multiple issues). You are very welcome to comment or propose solutions there though. |
Uh oh!
There was an error while loading. Please reload this page.
via
__init__.py
This is really quite bad behavior. If you feel like you really must do this, please use
append=True
The text was updated successfully, but these errors were encountered: