Bar format is str but should be Optional[str] in ignite.contrib.handlers.tqdm_logger.ProgressBar

[yohan-pg]

🐛 Bug description

ignite.contrib.handlers.tqdm_logger.ProgressBar can accept None in the bar_format initialization argument (as documented), but this is not reflected in the type annotation.

ignite/ignite/contrib/handlers/tqdm_logger.py

Line 120 in 82a929d

bar_format: str = "{desc}[{n_fmt}/{total_fmt}] {percentage:3.0f}%|{bar}{postfix} [{elapsed}<{remaining}]",

[vfdev-5]

Thanks for catching that @yohan-pg , would you like to send a fix ?
So, it should be

bar_format: Union[str, None] = "{desc}[{n_fmt}/{total_fmt}] {percentage:3.0f}%|{bar}{postfix} [{elapsed}<{remaining}]"

[Davidportlouis]

@vfdev-5 I would like to work on this fix

[vfdev-5]

Thanks @Davidportlouis , assigned it to you