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.

Sign up for GitHub

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

Jump to bottom

Setting RERUN=ON results in duplicate terminal output #1649

Closed
roym899 opened this issue Mar 21, 2023 · 1 comment
Closed

Setting RERUN=ON results in duplicate terminal output #1649

roym899 opened this issue Mar 21, 2023 · 1 comment
Labels
😤 annoying Something in the UI / SDK is annoying to use 🐍 Python API Python logging API

Comments

@roym899
Copy link
Collaborator

roym899 commented Mar 21, 2023

Describe the annoyance
Very low priority, but setting RERUN=ON results in the following output:

2023-03-21T13:56:46.150409Z  INFO re_sdk: Rerun Logging is enabled by the 'RERUN' environment variable.
2023-03-21T13:56:46.247752Z  INFO re_sdk: Rerun Logging is enabled by the 'RERUN' environment variable.

To Reproduce

  1. export RERUN=ON
  2. Run any rerun script that logs something.

Expected behavior

2023-03-21T13:56:46.150409Z  INFO re_sdk: Rerun Logging is enabled by the 'RERUN' environment variable.

Desktop (please complete the following information):

  • OS: Ubuntu 20.04
@roym899 roym899 added 👀 needs triage This issue needs to be triaged by the Rerun team 😤 annoying Something in the UI / SDK is annoying to use labels Mar 21, 2023
@roym899 roym899 changed the title Setting RERUN=ON results in duplicate terminal output Setting RERUN=ON results in duplicate terminal output Mar 21, 2023
@emilk emilk added 🐍 Python API Python logging API and removed 👀 needs triage This issue needs to be triaged by the Rerun team labels Mar 21, 2023
@emilk
Copy link
Member

emilk commented Mar 21, 2023

I'm 99% certain this is coming from two different Python processes. So this only happens on Python, with spawn. Both processes detect the RERUN environment variable, but since the second process is not an SDK library, it shouldn't really care about this and log it.

@jleibs jleibs mentioned this issue Apr 4, 2023
Merged
2 tasks
@jleibs jleibs closed this as completed Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
😤 annoying Something in the UI / SDK is annoying to use 🐍 Python API Python logging API
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@emilk @jleibs @roym899