Skip to content

Conversation

@phaiting
Copy link
Contributor

Summary

There are e2e test failures that is related with ModuleAdd.pte, before we fully fixed that, let's add the @ignore

See the failure logs:
10-17 21:45:28.303 2107 2284 E AndroidRuntime: FATAL EXCEPTION: Thread-101
10-17 21:45:28.303 2107 2284 E AndroidRuntime: Process: org.pytorch.executorch.test, PID: 2107
10-17 21:45:28.303 2107 2284 E AndroidRuntime: org.pytorch.executorch.ExecutorchRuntimeException$ExecutorchInvalidArgumentException: [Executorch Error 0x12] Invalid argument: Execution failed for method: forward

Test plan

  • verified on local android emulator
  • run via CI

10-17 21:45:28.303  2107  2284 E AndroidRuntime: FATAL EXCEPTION: Thread-101
10-17 21:45:28.303  2107  2284 E AndroidRuntime: Process: org.pytorch.executorch.test, PID: 2107
10-17 21:45:28.303  2107  2284 E AndroidRuntime: org.pytorch.executorch.ExecutorchRuntimeException$ExecutorchInvalidArgumentException: [Executorch Error 0x12] Invalid argument: Execution failed for method: forward
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 17, 2025

🔗 Helpful Links

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

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

❌ 1 New Failure, 6 Pending

As of commit 2e117e6 with merge base f4d801a (image):

NEW FAILURE - The following job has failed:

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 Oct 17, 2025
@phaiting
Copy link
Contributor Author

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Oct 17, 2025
@phaiting
Copy link
Contributor Author

The run-emulator run fails is expected because there are two test cases for llm is broken, this change will make the run-emulator to show the true e2e status.

@phaiting phaiting merged commit b858946 into main Oct 17, 2025
146 of 149 checks passed
@phaiting phaiting deleted the e2e-jobs branch October 17, 2025 23:47
@kirklandsign
Copy link
Contributor

Let's fix the other two?

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.

2 participants