Skip to content
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.

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

Render nested progress bars in the same container in IPython. #1504

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

UFO-101
Copy link

@UFO-101 UFO-101 commented Aug 27, 2023

Fixes rendering issues in VSCode: microsoft/vscode-jupyter#9397.

BEFORE:

Screen.Recording.2023-08-27.at.17.44.10.mov

AFTER:

Screen.Recording.2023-08-27.at.17.40.56.mov

Yes it's unclear who's fault this is, but it doesn't seem like anyone else is going to fix it anytime soon and this way seems fine anyway.

@ringohoffman
Copy link

@casperdcl is this reviewable? I'd really like to consume this fix for VS Code

@eiva
Copy link

eiva commented Dec 28, 2023

Any way to make it merged to mainline? It is pretty annoying visual bug which pollutes whole notebook in just few minutes...

@amunozj
Copy link

amunozj commented Jan 16, 2024

Any news on this? it would be amazing not to have this bug!

@wzp1229
Copy link

wzp1229 commented Feb 14, 2024

Still no news? Seems that it has been stuck in review for quite a while by now...

@PhilipMathieu
Copy link

+1 This should be merged as soon as reasonably possible

…endering issues in VSCode. See issue: microsoft/vscode-jupyter#9397.

Also fixes leave=None in IPython. As per the documentation, setting leave=None should leave iff position=0, but currently it is just equivalent to leave=False.
@UFO-101
Copy link
Author

UFO-101 commented Apr 28, 2024

I've released a fork of tqdm with this fix on PyPI: https://pypi.org/project/vscode-tqdm/

@UFO-101
Copy link
Author

UFO-101 commented Apr 28, 2024

But yeah this should really just be merged.

@PhilipMathieu
Copy link

@UFO-101 Thanks for your work on this. Using your version until @casperdcl has an opportunity to review and hopefully approve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants