-
Notifications
You must be signed in to change notification settings - Fork 47
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
Print solver warning messages to Python console #1951
Comments
So currently, if there are warnings found in the log, a warning is printed saying that.
The idea behind this was not to have a large output if there are many warning messages in the log. If I understand correctly what you are proposing is instead to display them all? |
Yes essentially. It's one of those situations where we can't make everyone happy. Some people prefer to see all the warnings so they know if there are potential issues. Some people prefer to have less warnings so the console/notebook look cleaner. I think we should probably lean towards the former and users can always turn off the warning if they want. |
just checking in here, I wonder if one possible solution is to just modify the code generating the warning above to add the extra logic needed to grab the log (like the warning message says?) and then just display it? or is that overly simplistic? I guess I'm confused why we couldn't just do that |
I think we can do it, my question above was basically if there are 20 warnings do we want to display them all? This single warning instead tells you something may be off without being too verbose. |
maybe we can use consolidated logger? |
There are some warnings in the logs but they are not printed in the console. One example is the warning about "TE mode specified in the mode source but no TE modes are found". Users think having this kind of messages printed out would be helpful.
The text was updated successfully, but these errors were encountered: