Fix custom progress bar with huggingface_hub >= 1.1.0

[danxuliu]

Fixes #389

Starting with version 1.1.0, huggingface_hub passes a name argument to the custom progress bar. However, the base tqdm class rejects unknown arguments, so it needs to be removed before passsing it to its constructor (like done in the progress bar provided by huggingface_hub).

Besides that, since the same huggingface_hub change mentioned above 0 is initially passed as the total when creating the progress bar, so its use in the custom progress bar has been guarded to prevent a division by zero error.

Note, however, that I am not familiar at all with tqdm, huggingface_hub or nc_py_api, so there might be better fixes :-) (like showing an Initializing... message rather than just guarding the call to nc.set_init_status if total <= 0 or things like that).

[codecov]

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.65%. Comparing base (ef69321) to head (a722af0).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
nc_py_api/ex_app/integration_fastapi.py	0.00%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
- Coverage   94.72%   94.65%   -0.08%     
==========================================
  Files          45       45              
  Lines        5385     5389       +4     
==========================================
  Hits         5101     5101              
- Misses        284      288       +4     

Files with missing lines	Coverage Δ
nc_py_api/ex_app/integration_fastapi.py	23.07% <0.00%> (-0.56%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[kyteinsky]

thanks a lot! The last version I tested was 1.1.0 of hf_hub, looking up the changelog was smart :)
It should be fine like this without the initilising message, wdyt @oleksandr-nc ?

[kyteinsky]

works like a charm!

[danxuliu]

looking up the changelog was smart :)

It would have been, but I did not until you mentioned it 🙈 I directly dived into the code when I got the error 😅 Beauty of FOSS ;-)

[oleksandr-nc]

Thank you for this PR, looks good now

[oleksandr-nc]

@kyteinsky wdyt, is it worth releasing a new version with this fix?

[kyteinsky]

@oleksandr-nc yeah, let's have one. I suppose the model download feature might be used in many apps, for now we need this for live_transcription.