Skip to content

Conversation

@vhsdream
Copy link
Member

@vhsdream vhsdream commented Nov 4, 2025

✍️ Description

  • There's another OCR fix in this one, but not sure if it will fix the issues it has with OpenVINO.

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@github-actions github-actions bot added bugfix update script A change that updates a script labels Nov 4, 2025
@schneider-de-com
Copy link
Contributor

@vhsdream why this pr is a draft Version?

@MickLesk
Copy link
Member

MickLesk commented Nov 5, 2025

Because he Test? And the version is not published?

@vhsdream
Copy link
Member Author

vhsdream commented Nov 5, 2025

That's right. Merging this now would do nothing besides prevent people from 2.2.2 and below from being able to update to 2.2.2.

Once 2.2.3 is officially released, and I've gone over the release notes and changelog, and determined that there are no changes needed to the scripts, then it will go live.

@MickLesk
Copy link
Member

MickLesk commented Nov 5, 2025

Alt text

@vhsdream vhsdream marked this pull request as ready for review November 5, 2025 16:02
@vhsdream vhsdream requested a review from a team as a code owner November 5, 2025 16:02
@vhsdream
Copy link
Member Author

vhsdream commented Nov 5, 2025

Unfortunately it doesn't appear this addresses the piss-poor OCR performance with OpenVINO:
image

It barely uses the GPU at all, and there also might be an issue with a memory leak.

@MickLesk MickLesk merged commit 72391a9 into community-scripts:main Nov 5, 2025
2 checks passed
@vhsdream vhsdream deleted the v2.2.3 branch November 5, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants