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

Enh: Add launch mode with a pass-through option for stderr #398

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alcrene
Copy link
Contributor

@alcrene alcrene commented Apr 21, 2024

The motivation for this was to allow using tqdm progress bars in my scripts, since those use stderr to display progress. I imagine this might be useful in other situations as well, including other progress bar libraries.

This option to leave stderr as-is is propagated through a new stderr keyword argument to run().

To use this option, I added a new launch mode is called SerialTqdmLaunchMode, which uses this new argument to disable stderr capture. Arguably a more generic may be more correct, but for now I went with this because

  • It’s relatively short (just four extra letters);
  • It’s more easily discoverable: not everyone will know that a tool like tqdm works by sending data through stderr, so with something like SerialPassStderrLaunchMode, it may not be immediately obvious that them that they should use this.

On the flip side, there may be an advantage to being explicit that stderr will not be captured with this launch mode. All that to say that I’m far from wedded to this name.

Adds the SerialTqdmLaunchMode, which disables stderr capture.
This option to leave stderr as-is is propagated through a new keyword argument
to `run()`.
@alcrene alcrene changed the title Add launch mode with a pass-through option for stderr Enh: Add launch mode with a pass-through option for stderr Apr 21, 2024
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.

1 participant