-
Notifications
You must be signed in to change notification settings - Fork 3
Add OutputConsole widget for displaying stdout/err, logging messages and tracebacks #43
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This helps when developing for the stream or console classes if you cause an exception you might get no traceback printed to debug otherwise.
Also ensure that it writes unix newlines on windows
Restore StreamHandlerHelper removed in previous commit by moving it into the more appropriate location of stream.install_to_std.
- This allows them to write their messages even if they wouldn't normally. - Also remove legacy unused outputPipe. Instead add a callback to preditor.active.
This prevents all instances of the class sharing the same variable if modified without using the setter.
- Consolidate truncate code under preditor.utils.Truncate. - Move result printing into the Console class inside executeString.
47bcf04 to
17bfb4e
Compare
Instead of stdout/err redirection. This can be configured using QtDesigner.
This improves the QtDesigner plugin behavior
This logs the inputs and return of the decorated function
This prevents cases where the hand cursor override gets stuck on if the mouse is quickly moved to another widget. It also seems to be processed faster.
1. Assume its encoded as utf-8 unless that causes an encoding error 2. Switch from chardet to charset-normalizer for encoding detection if otherwise detected Add a testing framework to verify that character encoding is resolved correctly.
4c6758f to
36d5c15
Compare
36d5c15 to
3205ed0
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rework the
ConsolePrEditclass into a base classe andOutputConsole. OutputConsole can be used in tools and UI's to show output from any combination of stdout, stderr, python logging and raised tracebacks. You can use multiple of them at once and close and open them without affecting the main ConsolePrEdit.Other improvements:
sys.__stderr__so you can debug problems with PrEditor.Checklist
Types of Changes