[PR #25604] fix(ocr): intel gpu memory leak fix #18310

Open
opened 2026-02-05 16:36:16 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/25604
Author: @savely-krasovsky
Created: 1/28/2026
Status: 🔄 Open

Base: mainHead: main


📝 Commits (10+)

  • 87cbb1a feat: update intel compute driver
  • db79af4 Merge pull request #2 from savely-krasovsky/intel-update
  • 1f0545e Merge branch 'main' into intel-update
  • 2f4ed87 fix: trying to fix memory leak
  • 234dbf3 Merge pull request #3 from savely-krasovsky/openvino-memory-leak
  • bd9223c feat: static size detection and static width recognition
  • 599e5e1 Merge branch 'main' into main
  • 7a2d11f Merge branch 'main' into main
  • 43f1b9a Merge branch 'main' into main
  • bfae05e Merge branch 'main' into main

📊 Changes

4 files changed (+104 additions, -14 deletions)

View changed files

📝 .editorconfig (+3 -0)
📝 machine-learning/immich_ml/config.py (+2 -0)
📝 machine-learning/immich_ml/models/ocr/detection.py (+71 -14)
📝 machine-learning/immich_ml/models/ocr/recognition.py (+28 -0)

📄 Description

Description

Fixes # (issue)

How Has This Been Tested?

  • Test A
  • Test B

Screenshots (if appropriate)

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

Helped me to write valid Python code. I am not a python developer.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/25604 **Author:** [@savely-krasovsky](https://github.com/savely-krasovsky) **Created:** 1/28/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (10+) - [`87cbb1a`](https://github.com/immich-app/immich/commit/87cbb1ae63e976b91cc4c0b698ca7df008620475) feat: update intel compute driver - [`db79af4`](https://github.com/immich-app/immich/commit/db79af49f3fc205164d4e2392a42f470d7a2f739) Merge pull request #2 from savely-krasovsky/intel-update - [`1f0545e`](https://github.com/immich-app/immich/commit/1f0545ef2bb95ccad11e4427ef434d9e8c80c156) Merge branch 'main' into intel-update - [`2f4ed87`](https://github.com/immich-app/immich/commit/2f4ed87624ae481282c48a79837456a5b37ca55e) fix: trying to fix memory leak - [`234dbf3`](https://github.com/immich-app/immich/commit/234dbf37956088b030e108c3ed74f12d7189b382) Merge pull request #3 from savely-krasovsky/openvino-memory-leak - [`bd9223c`](https://github.com/immich-app/immich/commit/bd9223c8df029d409751ec379d6cd4e1dad7e21b) feat: static size detection and static width recognition - [`599e5e1`](https://github.com/immich-app/immich/commit/599e5e1c757cd2600584d89266fc7259a17abaac) Merge branch 'main' into main - [`7a2d11f`](https://github.com/immich-app/immich/commit/7a2d11fb61fc9c678eee58c3922c0124a02254c6) Merge branch 'main' into main - [`43f1b9a`](https://github.com/immich-app/immich/commit/43f1b9acf17e239202e9b279d5b35f84258fe7bf) Merge branch 'main' into main - [`bfae05e`](https://github.com/immich-app/immich/commit/bfae05e0d37be70a8d1074b60aa7309a34fb46a3) Merge branch 'main' into main ### 📊 Changes **4 files changed** (+104 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `.editorconfig` (+3 -0) 📝 `machine-learning/immich_ml/config.py` (+2 -0) 📝 `machine-learning/immich_ml/models/ocr/detection.py` (+71 -14) 📝 `machine-learning/immich_ml/models/ocr/recognition.py` (+28 -0) </details> ### 📄 Description ## Description <!--- Describe your changes in detail --> <!--- Why is this change required? What problem does it solve? --> <!--- If it fixes an open issue, please link to the issue here. --> Fixes # (issue) ## How Has This Been Tested? <!-- Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration --> - [ ] Test A - [ ] Test B <details><summary><h2>Screenshots (if appropriate)</h2></summary> <!-- Images go below this line. --> </details> <!-- API endpoint changes (if relevant) ## API Changes The `/api/something` endpoint is now `/api/something-else` --> ## Checklist: - [ ] I have performed a self-review of my own code - [ ] I have made corresponding changes to the documentation if applicable - [ ] I have no unrelated changes in the PR. - [ ] I have confirmed that any new dependencies are strictly necessary. - [ ] I have written tests for new code (if applicable) - [ ] I have followed naming conventions/patterns in the surrounding code - [ ] All code in `src/services/` uses repositories implementations for database calls, filesystem operations, etc. - [ ] All code in `src/repositories/` is pretty basic/simple and does not have any immich specific logic (that belongs in `src/services/`) ## Please describe to which degree, if any, an LLM was used in creating this pull request. Helped me to write valid Python code. I am not a python developer. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 16:36:16 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#18310