Skip to content

add verbose as default jupyter notebooks#831

Merged
rickwierenga merged 4 commits intoPyLabRobot:mainfrom
tibisabau:794-add-verbose-default-jupyter
Jan 22, 2026
Merged

add verbose as default jupyter notebooks#831
rickwierenga merged 4 commits intoPyLabRobot:mainfrom
tibisabau:794-add-verbose-default-jupyter

Conversation

@tibisabau
Copy link
Copy Markdown
Contributor

Description

Implemented automatic verbose logging for PyLabRobot when running in Jupyter notebook environments. Added a helper function _is_running_in_jupyter() that detects if the code is executing within a Jupyter/IPython session by checking for the get_ipython built-in. Modified the configure() function to automatically enable verbose mode by calling verbose(True) when a Jupyter environment is detected. This enhancement improves the user experience by providing immediate feedback and logging output in interactive notebook environments without requiring manual configuration.

Closes #794.

The code in this pull request was generated by GitHub Copilot with the Claude Sonnet 4.5 model.

Checklist if Applicable

  • The tests passed – make test (650 tests passed, 0 failures)
  • Linting passed – make lint, make format-check, make format
  • Documentation has been added
  • CHANGELOG.md has been updated

tibisabau and others added 4 commits January 22, 2026 17:39
The previous check only verified that get_ipython exists, which is true
for any IPython session. Now properly checks for ZMQInteractiveShell
which is specific to Jupyter notebooks/lab.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Wrap comparison in bool() to satisfy mypy's no-any-return check.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@rickwierenga rickwierenga merged commit 95b0d20 into PyLabRobot:main Jan 22, 2026
10 checks passed
@rickwierenga
Copy link
Copy Markdown
Member

thanks @tibisabau !

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.

we should be verbose by default when in a Jupyter notebook

2 participants