Skip to content
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

Modified connection-flask.py so that each print statement gets flushe… #11

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

bieryAtFnal
Copy link
Contributor

…d immediatedly (this helps when debugging), and added a printout of each published UID.

In my experience, the output from the debug print statements (without my flush=True addition) can be very confusing because they seem to appear at somewhat random places in the ConnSvc log file.

I can provide instructions for how to test this, if that is useful.

…d immediatedly (this helps when debugging), and added a printout of each published UID.
Copy link
Member

@eflumerf eflumerf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, minimal_system_quick_test still passes

@bieryAtFnal bieryAtFnal merged commit 0c85cb0 into develop Jan 31, 2025
1 check passed
@bieryAtFnal bieryAtFnal deleted the kbiery/flush_printout_etc branch January 31, 2025 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants