Stop gathering yolo26 for the portable CI matrix - #21618
Merged
Merged
Conversation
yolo26 has never passed on the portable backend since it was added, so the gathered test-models-linux job for it is permanently red and produces no signal. Skip it in gather_test_models.py; the model itself is unchanged and its OpenVINO and XNNPACK paths are unaffected.
JacobSzwejbka
approved these changes
Aug 6, 2026
This PR needs a
|
Contributor
Author
|
Good call, filed #21621 and referenced it from the skip so it does not become permanent by accident. |
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.
test-models-linux (yolo26, portable)has been red since yolo26 was added and has never had a green run, so it produces no signal and just adds noise to every periodic run.yolo26 is registered in the shared model registry (
examples/models/__init__.py), andgather_test_models.pyexpands every registry entry across theportableandxnnpackbackends. yolo26 is not inMODEL_NAME_TO_OPTIONS, so the xnnpack variant is already skipped, which means the portable job is the only one this gather step ever produced.This change skips yolo26 in the gather step. Verified locally by running the script before and after: the gathered matrix goes from 53 records to 52, and the only record removed is
The model itself, its example code, and its OpenVINO / XNNPACK / RISC-V paths are all untouched.
@daniil-lyakhov flagging you since you added yolo26 in #18583. If you do want portable coverage for it, the fix is to make it export cleanly on the portable backend and then drop this skip. Happy to revert this if you pick it up.