Skip to content

Stop gathering yolo26 for the portable CI matrix - #21618

Merged
shoumikhin merged 2 commits into
mainfrom
oncall/drop-yolo26-portable-ci
Aug 6, 2026
Merged

Stop gathering yolo26 for the portable CI matrix#21618
shoumikhin merged 2 commits into
mainfrom
oncall/drop-yolo26-portable-ci

Conversation

@shoumikhin

Copy link
Copy Markdown
Contributor

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), and gather_test_models.py expands every registry entry across the portable and xnnpack backends. yolo26 is not in MODEL_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

{'build-tool': 'cmake', 'model': 'yolo26', 'backend': 'portable', 'runner': 'linux.2xlarge', 'timeout': 90}

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.

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.
Copilot AI lite review requested due to automatic review settings August 6, 2026 17:21
@pytorch-bot

pytorch-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/21618

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

❌ 3 New Failures, 17 Cancelled Jobs, 29 Pending, 2 Unclassified Failures

As of commit 7d863cf with merge base 85ade9f (image):

NEW FAILURES - The following jobs have failed:

UNCLASSIFIED FAILURES - DrCI could not classify the following jobs because the workflow did not run on the merge base. The failures may be pre-existing on trunk or introduced by this PR:

CANCELLED JOBS - The following jobs were cancelled. Please retry:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 6, 2026

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@digantdesai digantdesai left a comment

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.

Add a TODO issue?

@shoumikhin

Copy link
Copy Markdown
Contributor Author

Good call, filed #21621 and referenced it from the skip so it does not become permanent by accident.

Copilot AI review requested due to automatic review settings August 6, 2026 17:56

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@shoumikhin shoumikhin added the release notes: none Do not include this in the release notes label Aug 6, 2026
@shoumikhin
shoumikhin merged commit 99924e4 into main Aug 6, 2026
314 of 339 checks passed
@shoumikhin
shoumikhin deleted the oncall/drop-yolo26-portable-ci branch August 6, 2026 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants