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
improve params errors #2437
improve params errors #2437
Conversation
still needs work, I forgot about the warnings we send to stderr like "rung does not cross both rails" and such |
@kaalleen Do you think this is ready to go? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love the possibility to see the error message within the params and simulator dialogs.
Seems to be working in Linux, macOS and Windows.
Some error messages are hard to read when they are a longer. Therefore I'd like if we would wrap the text if it is too long. Especially in the simulator window users wouldn't be able to read to whole text if their screens are small.
Somewhere along the line, the thing that shows a red error message in the Params window when exceptions occur stopped working. This PR fixes that and makes it way more useful. Now, instead of telling them to close the simulator to see the error, the error is shown directly in the Params window.
This way they don't get a spam of errors on stderr that all show up at the end.