Skip to content

ITEP-81114: Update set of models used in model downloader script#1022

Merged
saratpoluri merged 12 commits intomainfrom
ITEP-81114
Feb 19, 2026
Merged

ITEP-81114: Update set of models used in model downloader script#1022
saratpoluri merged 12 commits intomainfrom
ITEP-81114

Conversation

@MikolajKasprzak
Copy link
Copy Markdown
Contributor

📝 Description

Update set of models used in model downloader script, remove unused ones

✨ Type of Change

Select the type of change your PR introduces:

  • 🐞 Bug fix – Non-breaking change which fixes an issue
  • 🚀 New feature – Non-breaking change which adds functionality
  • 🔨 Refactor – Non-breaking change which refactors the code base
  • 💥 Breaking change – Changes that break existing functionality
  • 📚 Documentation update
  • 🔒 Security update
  • 🧪 Tests
  • 🚂 CI

🧪 Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • ✅ Tested manually
  • 🤖 Ran automated end-to-end tests

✅ Checklist

Before submitting the PR, ensure the following:

  • 🔍 PR title is clear and descriptive
  • 📝 For internal contributors: If applicable, include the JIRA ticket number (e.g., ITEP-123456) in the PR title. Do not include full URLs
  • 💬 I have commented my code, especially in hard-to-understand areas
  • 📄 I have made corresponding changes to the documentation
  • ✅ I have added tests that prove my fix is effective or my feature works

Copy link
Copy Markdown
Contributor

@tdorauintc tdorauintc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review in progress

'text-recognition-resnet-fc',
'text-recognition-0012'
]
_OCR_MODELS = _DEFAULT_MODELS
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
_OCR_MODELS = _DEFAULT_MODELS

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file needs to be cleaned up. Lot of unused variables.

@saratpoluri saratpoluri enabled auto-merge (squash) February 19, 2026 07:28
@saratpoluri saratpoluri merged commit d56e6f3 into main Feb 19, 2026
30 checks passed
@saratpoluri saratpoluri deleted the ITEP-81114 branch February 19, 2026 12:00
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.

4 participants