Skip to content

⚡ Bolt: [performance improvement] Pass verbose=False to YOLO predict calls#22

Draft
kingkillery wants to merge 1 commit into
mainfrom
bolt/optimize-yolo-predict-17640584995584275623
Draft

⚡ Bolt: [performance improvement] Pass verbose=False to YOLO predict calls#22
kingkillery wants to merge 1 commit into
mainfrom
bolt/optimize-yolo-predict-17640584995584275623

Conversation

@kingkillery
Copy link
Copy Markdown
Owner

@kingkillery kingkillery commented Apr 15, 2026

💡 What: Added verbose=False to the Ultralytics model.predict() calls in commonforms/inference.py.

🎯 Why: By default, YOLO inference prints details to stdout for every image. When processing documents with many pages, the string formatting and synchronous I/O operations block the main thread, adding unnecessary overhead.

📊 Impact: Reduces synchronous stdout blocking overhead during prediction, leading to slightly faster document processing times (especially on multi-page PDFs) without affecting model output or correctness.

🔬 Measurement: Run a multi-page PDF processing task through the prepare_form pipeline. You will observe no print spam to standard output, reducing processing time and cleaning up execution logs. All standard functionality test continue to pass identically.


PR created automatically by Jules for task 17640584995584275623 started by @kingkillery

Summary by CodeRabbit

  • Chores
    • Reduced verbose output from model predictions to improve log clarity during processing.

…calls

Disabled verbose logging on Ultralytics model.predict() calls to prevent synchronous I/O blocking during inference loops.

Co-authored-by: kingkillery <200727508+kingkillery@users.noreply.github.com>
@google-labs-jules
Copy link
Copy Markdown

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a868b583-6921-41d2-8f62-1321f17a6a4c

📥 Commits

Reviewing files that changed from the base of the PR and between e00f2d1 and 2ae5598.

📒 Files selected for processing (1)
  • commonforms/inference.py

Walkthrough

Added verbose=False parameter to model prediction calls within the FFDNetDetector.extract_widgets method. Changes apply to both ONNX and batched prediction paths without altering existing logic or control flow.

Changes

Cohort / File(s) Summary
Verbose Suppression
commonforms/inference.py
Added verbose=False to two self.model.predict(...) calls in FFDNetDetector.extract_widgets—one for ONNX per-page prediction and one for batched prediction with augmentation. Suppresses verbose output without modifying logic or return structures.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A quiet prediction, serene and refined,
No verbose chatter cluttering the mind,
Two words whispered soft: "Hush, be still,"
The models obey with focused will,
Cleaner logs, a joy to behold! 🌙

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: passing verbose=False to YOLO predict calls for performance improvement. It accurately reflects the primary modification in the changeset.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bolt/optimize-yolo-predict-17640584995584275623

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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