Repository navigation
Remove YOLOv5, YOLOv7, YOLOv9, and YOLOv10 upload support - #572
Merged
Merged
Conversation
Co-authored-by: OpenAI Codex <codex@openai.com>
Contributor
Author
|
Note This comment was NOT written by Kai. It was written by Kai's Claude and posted through Kai's GitHub account on this PR. /jarbas review |
mkaic
marked this pull request as ready for review
October 28, 2026 20:49
Contributor
Author
|
/jarbas review |
leeclemnet
reviewed
Oct 30, 2026
leeclemnet
left a comment
leeclemnet
left a comment
Contributor
There was a problem hiding this comment.
LGTM but needs a changelog entry
Co-authored-by: OpenAI Codex <codex@openai.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
leeclemnet
approved these changes
Oct 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
Code and PR description are LLM-written. If this PR has been marked Ready For Review, code has
been reviewed and tested in staging by Kai
Diff breakdown
Approximate added and removed lines.
Description
Remove YOLOv5, YOLOv7, YOLOv9, and YOLOv10 from custom-weight upload support. Packaging rejects these families before importing Torch or loading checkpoints. Delete the legacy packaging branches and replace retired upload examples and CLI help. Dataset formats and inference for existing models remain supported.
Companion API enforcement: roboflow #18676.
Related documentation: roboflow/docs #453.
Type of change
Breaking change: remove legacy upload support.
Testing
Ruff formatting, lint, Python syntax, and diff checks passed.
Tested together with the companion API, converter, and SDK PRs on US staging (
kaiplace) on 2026-10-28. Comparisons use local CPU runtimes and the pre-retirement exports.RF-DETR keypoint PyTorch and YOLO11 TFJS remain non-executable in both the baseline and new exports. Existing YOLOv5 retains its registered-loader filename mismatch. No existing YOLOv7/9/10 staging models were available. Florence OCR samples return empty text. Browser Workflows and GPU execution were not tested.
Deployment
Deploy the API change first. Publish a package release through the existing release process; publishing a GitHub release triggers
Publish WorkFlow. No SDK release was published.