Skip to content

Use console tqdm to avoid notebook warnings#814

Draft
dangzitou wants to merge 1 commit into
mljar:masterfrom
dangzitou:openmeta/agent-657-tqdm-warning-in-notebook-1779667434702
Draft

Use console tqdm to avoid notebook warnings#814
dangzitou wants to merge 1 commit into
mljar:masterfrom
dangzitou:openmeta/agent-657-tqdm-warning-in-notebook-1779667434702

Conversation

@dangzitou

Copy link
Copy Markdown

Summary

Kaggle and Jupyter users can see a warning because the package imports tqdm from tqdm.autonotebook, which tries to select notebook mode automatically. This change replaces autonotebook usage with standard console-mode tqdm imports while preserving existing progress bar behavior, and adds regression coverage to prevent reintroducing the warning-causing import.

Changes

  • Replace tqdm.autonotebook imports in the AutoML training and tuning code with standard tqdm imports.
  • Keep existing tqdm call sites unchanged where standard tqdm supports the same iterable wrapping and progress bar options.
  • Add a regression test that scans Python files under supervised/ and fails if tqdm.autonotebook is referenced.

Validation

  • pytest tests/tests_utils/test_tqdm_imports.py is recommended for the new regression coverage and is still pending because no runnable commands were provided or executed.
  • pytest tests/tests_validation tests/tests_utils is recommended for related validation and is still pending because no runnable commands were provided or executed.
  • Manual verification in Kaggle or Jupyter that AutoML import and fit no longer emit the tqdm.autonotebook warning is still pending.

Risks

  • Notebook progress bars may render as console-style tqdm output instead of rich notebook widgets.
  • There may be additional tqdm.autonotebook references outside the initially identified files that need to be replaced.
  • The regression test scans source references and may need exclusions if documentation or comments intentionally mention tqdm.autonotebook.

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.

1 participant