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

Disable concurrent progress bars in Jupyter Notebooks #3890

Merged
merged 3 commits into from
May 28, 2024

Conversation

ibraheemdev
Copy link
Member

Summary

Resolves #3887 by disabling the new progress output when the JPY_SESSION_NAME environment variable is detected.

Test Plan

Fixed output in a Jupyter notebook:

image

@@ -201,6 +266,9 @@ impl uv_installer::DownloadReporter for DownloadReporter {
}

fn on_complete(&self) {
// Need an extra call to `set_message` here to fully clear avoid leaving ghost output
// in Jupyter notebooks.
self.reporter.root.set_message("");
Copy link
Member Author

@ibraheemdev ibraheemdev May 28, 2024

Choose a reason for hiding this comment

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

This bug existed before the new progress bars, where if the previous progress bar line was longer than the next one, it wouldn't be fully cleared. Seems like a bug in indicatif.

@charliermarsh
Copy link
Member

Can you also test this in Jupyter Lab? Same process but run jupyter lab in the command line instead of jupyter notebook. It's kind of the next-gen UI.

@charliermarsh charliermarsh added the bug Something isn't working label May 28, 2024
Copy link

codspeed-hq bot commented May 28, 2024

CodSpeed Performance Report

Merging #3890 will improve performances by 6.72%

Comparing ibraheemdev:jupyter-progress (99a1c0c) with main (47db418)

Summary

⚡ 1 improvements
✅ 12 untouched benchmarks

Benchmarks breakdown

Benchmark main ibraheemdev:jupyter-progress Change
wheelname_tag_compatibility[flyte-short-incompatible] 926.7 ns 868.3 ns +6.72%

@ibraheemdev
Copy link
Member Author

@charliermarsh seems to work there too:
image

@ibraheemdev ibraheemdev merged commit 038af6e into astral-sh:main May 28, 2024
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uv progress bars: long output in jupyterlab
2 participants