Skip to content

only print verbosity messages in a notebook#1921

Open
joshqsumner wants to merge 1 commit into
v5.0from
jupyterconfig-quite-workers
Open

only print verbosity messages in a notebook#1921
joshqsumner wants to merge 1 commit into
v5.0from
jupyterconfig-quite-workers

Conversation

@joshqsumner
Copy link
Copy Markdown
Contributor

Describe your changes
Only print verbosity messages if JupyterConfig object is in a notebook. This keeps you from potentially getting a ton of printed output that doesn't add any information.

Type of update
This is a feature enhancement.

Associated issues
Closes #1917

Additional context
None

For the reviewer
See this page for instructions on how to review the pull request.

  • PR functionality reviewed in a Jupyter Notebook
  • All tests pass
  • Test coverage remains 100%
  • Documentation tested
  • New documentation pages added to plantcv/mkdocs.yml
  • Changes to function input/output signatures added to updating.md
  • Code reviewed
  • PR approved

@joshqsumner joshqsumner added enhancement Enhancements to existing features good first issue/PR Suitable for a first time contributor/reviewer labels May 12, 2026
@deepsource-io
Copy link
Copy Markdown

deepsource-io Bot commented May 12, 2026

DeepSource Code Review

We reviewed changes in 38b01e4...29ed3a2 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python May 12, 2026 8:23p.m. Review ↗
Code coverage May 12, 2026 8:23p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
100%
[✓ above threshold]
99.9%
[▼ down 0.1% from main]
Python
100%
[✓ above threshold]
99.9%
[▼ down 0.1% from main]

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancements to existing features good first issue/PR Suitable for a first time contributor/reviewer ready to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant