One item worth consideration: by overriding the default color we may make the bar less visible on some systems. E.g. someone who uses a light-green terminal may have a hard time seeing a green progress bar. I don't think there's a combination of colors that will work on every system, so perhaps the solution could be to allow turning this behavior off somehow.

@sangeet2020, could you please do the modifications suggested by @anautsch?

Hi,
First of all, apologies for the big delay in progressing this PR.

I agree with all the suggestions @pplantinga and @anautsch mentioned. It's better to have an override that can suppress the colored progress bar and rather use the default color- white.

Here is the override

"--white_tqdm", 
default=False, 
action="store_true",
 help="Enable default (white) color for tqdm progress-bar. If this is "
 "not passed a multi-colored progress-bar is used.",

and within run_opt_defauls, I added

"white_tqdm": False,
 "tqdm_progressbar": {
    "train": "GREEN",
     "valid": "MAGENTA",
     "test": "CYAN",
},

so whenever, --white_tqdm is passed, it sets all colors to an empty string (empty string implies tqdm would use the default color )

# Force default color for tqdm progrressbar
if self.white_tqdm:
    self.tqdm_progressbar = dict.fromkeys(self.tqdm_progressbar, "")

Let me know if this is an acceptable optimal solution.

If you guys are okay with this, I shall further change the currently hard-coded colors to use the colors present in SpeechBrain's logo as mentioned by @anautsch (328ce7 and a darker 4c99e8). But I have a concern that most of us use a dark terminal, so a dark-color progress bar may not be our first choice. What do you think?

Hi @anautsch, is there any update on this one? Do you recommend any other color combination?

Thanks for your detailed thoughts.

• Default value tqdm_default_color=True. Changes made as suggested above.
• It is true that we are not capable to control all environments- be it a white terminal, jupyter, green console etc.; Tqdm adjusts the color of its progress bar automatically. Therefore, it's wiser to let tqdm use its default settings.

• In a way, having nice colors matching to the terminal settings must be a tqdm concern with a good amount of discussion done already. Is there more to it?

  • True, we should let tqdm handle color of its progress bar in an unknown terminal setting. I don't think we should put extra thought than this.
• I could not find any relevant documentation on setting a background color for tqdm progress bar.

In my opinion, we should make it simple- use the default colored progress bar, or use a multi-colored progress bar and if that's a problem- switch back to the default setting and let tqdm handle what color to use in particular that environment setting.

In my opinion, we should make it simple- use the default colored progress bar, or use a multi-colored progress bar and if that's a problem- switch back to the default setting and let tqdm handle what color to use in particular that environment setting.

Sounds good!

I ran a test with

python recipes/LibriSpeech/ASR/seq2seq/train.py recipes/LibriSpeech/ASR/seq2seq/hparams/train_BPE_1000.yaml --data_folder=tests/samples/ASR/ --train_csv=tests/samples/annotation/ASR_train.csv --valid_csv=tests/samples/annotation/ASR_train.csv --test_csv=[tests/samples/annotation/ASR_train.csv] --number_of_epochs=10 --skip_prep=True --data_folder_rirs=tests/tmp --device="cpu" --debug --tqdm_default_color

and the color did not change—when I took --tqdm_default_color out, the color did not change either.

Do you have a minimal example that I could use to verify it works?
Maybe the issue is on my side...

When I commented above, I thought it would be possible to provide a --tqdm_default_color=True or --tqdm_default_color=False - but since this is a flag, this is not possible. I'm still indecessive on what would be better (default: we take tqdm default or default: we change the colors).
// Maybe we are asking too much with having the color dict to be also handed over as a parameter via "--args=..." style. If it is not possible to pass on a dict, a list could do as well (it's still ordered data).

Hi,
I made a few changes- change of variable names for easy understanding. (Please pull again)

Basically, not passing any arguments would enable default setting for tqdm progress bar. While if --tqdm_colored_bar is passed, a colored progress bar is enabled. You could find two snap shot showing these setups.

• Default tqdm- would work as has always been; no colors
  

• colored tqdm- one would need to pass --tqdm_colored_bar to enable this.
  

Let me know if you find this suitable or you wish some changes.

Thank You

Looks good to me. Thank You SB Team.