Skip to content

TQDM workaround due to unresponsive maintainer - #1363

Merged
davidbrochart merged 3 commits into
ipython:mainfrom
Carreau:tqdmfix
Mar 21, 2025
Merged

TQDM workaround due to unresponsive maintainer#1363
davidbrochart merged 3 commits into
ipython:mainfrom
Carreau:tqdmfix

Conversation

@Carreau

Copy link
Copy Markdown
Member

tqdm has an incorrect detection of ZMQInteractiveShell when launch via a scheduler that bypass IPKernelApp. Think of JupyterHub cluster spawners and co.

As of end of Feb 2025, the maintainer has been unresponsive for 5 months, to our fix, so we implement a workaround. I don't like it but we have few other choices.

See tqdm/tqdm#1628

tqdm has an incorrect detection of
ZMQInteractiveShell when launch via a scheduler
that bypass IPKernelApp. Think of JupyterHub
cluster spawners and co. As of end of Feb 2025,
the maintainer has been unresponsive for 5 months,
to our fix, so we implement a workaround. I don't
like it but we have few other choices. See
`https://github.com/tqdm/tqdm/pull/1628`

@krassowskikrassowski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see that even once the maintainer of tqdm has some more time, they can take a while to respone as there is a a few upstream PRs related to improving notebook autodetection:

Not sure if we can provide any feedback on the other PRs, but if there are any thoughts we could share to help with review upstream, this might help to move it forward.

Comment threadipykernel/zmqshell.py Outdated
@Carreau

Copy link
Copy Markdown
MemberAuthor

To backport.

@CarreauCarreau added this to the 6.5 milestone Feb 26, 2025
Co-authored-by: Min RK <benjaminrk@gmail.com>
Comment threadipykernel/zmqshell.py Outdated
@Carreau

Copy link
Copy Markdown
MemberAuthor

I hope this will get merged one day.

@davidbrochart
davidbrochart merged commit 9008dbb into ipython:mainMar 21, 2025
ianthomas23 pushed a commit to ianthomas23/ipykernel that referenced this pull request Jul 14, 2025
Co-authored-by: Min RK <benjaminrk@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Carreau@minrk@davidbrochart@krassowski