Thanks to visit codestin.com
Credit goes to github.com

Skip to content

set default tqdm.disable from os.environ #950

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
Closed
kaivalyar wants to merge 1 commit into from
Closed

set default tqdm.disable from os.environ #950

kaivalyar wants to merge 1 commit into from

Conversation

@kaivalyar
Copy link

@kaivalyar kaivalyar commented Apr 26, 2020 •
edited
Loading

Allow setting the tqdm.disable option from the environment variables. The default value of False is unchanged, as is the expected behavior with None.

Resolves issues laid out it #619 #614 #612. Similar to the request in #370.

  • I have marked all applicable categories:
    • exception-raising fix
    • visual output fix
    • documentation modification
    • new feature
  • If applicable, I have mentioned the relevant/related issue(s)

Less important but also useful:

  • I have visited the source website, and in particular
    read the known issues
  • I have searched through the issue tracker for duplicates
  • I have mentioned version numbers, operating system and
    environment, where applicable: 
    import tqdm, sys
print(tqdm.__version__, sys.version, sys.platform)

@kaivalyar
set default tqdm.disable from os.environ
89521ea 
Allow setting the tqdm.disable option from the environment variables. The default value of `False` is unchanged, as is the expected behavior with `None`.
@kaivalyar kaivalyar requested a review from casperdcl as a code owner April 26, 2020 23:45
@jarednielsen
Copy link

jarednielsen commented May 23, 2020

This would be helpful for my use case, where I'm using a third-party library which has a tqdm bar I'd like to disable but can't modify the library code itself easily.

@kaivalyar
Copy link
Author

kaivalyar commented May 23, 2020

Yes, this is exactly where I ran into this issue. I'm using pgmpy, which displays it's loops using the default tqdm behavior, which I would like to disable.

@jarednielsen
Copy link

jarednielsen commented May 23, 2020

Looks like this PR has been open for quite some time. @casperdcl Any thoughts on this change?

@zeeraktalat zeeraktalat mentioned this pull request Jun 11, 2020
Closed
casperdcl added a commit that referenced this pull request Jun 25, 2020
@casperdcl
add contrib.bells
8d721b2 
All the bells & whistles!

- related #614
- realted #950
@Grokzen
Copy link

Grokzen commented Oct 26, 2020

A simple use-case for this option is that we are running a big set of code where we have both support to run the exact same code as a cli tool from a users computer and in a hosted environment inside celery. And when we run it in celery, we do not want our logs to be polluted with tdqm loggin output. This option would make it super easy to disable all tdqm logging as we have it in a lot of places in our code and we would just configure our production servers with this environment variable and we are all set.

@Grokzen Grokzen mentioned this pull request Oct 27, 2020
Closed
8 tasks
@BastianZim BastianZim mentioned this pull request Dec 25, 2020
Closed
@nss350 nss350 mentioned this pull request Oct 12, 2021
Closed
@dilpath dilpath mentioned this pull request Jan 11, 2022
Closed
@Mahdi-Hosseinali
Copy link

Mahdi-Hosseinali commented May 31, 2022

Any updates on this PR? This is very useful when running tests and in production.

@mrclary mrclary mentioned this pull request Aug 10, 2022
Closed
@ZanSara ZanSara mentioned this pull request Jun 12, 2023
Closed
casperdcl added a commit that referenced this pull request Aug 5, 2023
@casperdcl
env var override support
0c6de06 
- fixes #370, fixes #612, fixes #619, fixes #1318
- closes #950, closes #1061
casperdcl added a commit that referenced this pull request Aug 8, 2023
@casperdcl
env var override support
e9dd1ef 
- fixes #370, fixes #612, fixes #619, fixes #1318
- closes #950, closes #1061
@casperdcl casperdcl mentioned this pull request Aug 8, 2023
Merged
casperdcl added a commit that referenced this pull request Aug 8, 2023
@casperdcl
env var override support
52d7812 
- fixes #370, fixes #612, fixes #619, fixes #1318
- closes #950, closes #1061
@casperdcl casperdcl self-assigned this Aug 9, 2023
@casperdcl casperdcl added duplicate 🗐 Seen it before p3-enhancement 🔥 Much new such feature c3-small 🕒 Complexity medium labels Aug 9, 2023
@casperdcl
Copy link
Member

casperdcl commented Aug 9, 2023

Thanks everyone for your patience! Just released tqdm==4.66.0 which includes support for env var overrides. I'm using TQDM_MININTERVAL=5 in most of my CI jobs now :)

@td-anne td-anne mentioned this pull request Sep 25, 2023
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@casperdcl casperdcl Awaiting requested review from casperdcl casperdcl is a code owner

Assignees

@casperdcl casperdcl

Labels

c3-small 🕒 Complexity medium duplicate 🗐 Seen it before p3-enhancement 🔥 Much new such feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kaivalyar @jarednielsen @Grokzen @Mahdi-Hosseinali @casperdcl