-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
MAINT: Correct odeint messages #4662
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
Conversation
Well, it wasn't a good idea to change global settings indeed.
In the original issue #4565 it was mentioned that 'There should also be an option to "silence" it.' |
I think it's best to make a new commit and squash them into one, |
odeint is writing messages to standard output. Change to use warnings for messages.
Done |
Not sure whether |
You are welcome! Thanks for help @ev-br! |
You don't have to, but it's indeed a good idea. I usually delete all merged branches to prevent my own repo becoming too messy. |
@vermelha if you include |
@rgommers, thanks for advices! |
accordind to issue #4565 odeint is writing messages to standard output. Change to use warnings for messages.