Skip to content

Conversation

@Erik-Lundell
Copy link
Collaborator

@Erik-Lundell Erik-Lundell commented Feb 10, 2025

Add markers to flaky tests
Check that reference model was run in TosaReferenceModelDispatch
If the if statment looking for executorch_call_delegates fails,
the delegate can run the model in eager mode and still produce correct results
without the user noticing. We need to make sure we actually run the reference model.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

If the if statment looking for executorch_call_delegates fails,
the delegate can run the model in eager mode and still produce correct results
without the user noticing. We need to make sure we actually run
the reference model.

Change-Id: I5fe070338e65e3e253cd319202b61fb4b389fde4
Change-Id: I27404d649c5e9945b319857f1f008cbfe06d20aa
@pytorch-bot
Copy link

pytorch-bot bot commented Feb 10, 2025

🔗 Helpful Links

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

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

❌ 2 Cancelled Jobs

As of commit 57bc715 with merge base 524ec78 (image):

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

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

@facebook-github-bot facebook-github-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 Feb 10, 2025
@Erik-Lundell Erik-Lundell added ciflow/trunk partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing and removed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Feb 10, 2025
@facebook-github-bot facebook-github-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 Feb 10, 2025
@zingo
Copy link
Collaborator

zingo commented Feb 10, 2025

MacOs fails unrelated

@zingo zingo merged commit 5aa0ee2 into pytorch:main Feb 10, 2025
107 of 111 checks passed
@Erik-Lundell Erik-Lundell deleted the fix_tosa_dispatch branch July 31, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants