Skip to content

Conversation

martinlsm
Copy link
Collaborator

@martinlsm martinlsm commented Apr 4, 2025

If the user does not provide input files with the --llama_inputs argument, the test is not run. Change the return statement into a pytest.skip so that the test output show a skip rather than a pass.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

If the user does not provide input files with the --llama_inputs
argument, the test is not run. Change the return statement into a
pytest.skip so that the test output show a skip rather than a pass.

Change-Id: I5a1836cb9a16dc5b3fd43e25f1586d901cc8e8f9
@martinlsm martinlsm requested a review from digantdesai as a code owner April 4, 2025 14:18
Copy link

pytorch-bot bot commented Apr 4, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit 3b0c04d with merge base cfca108 (image):

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

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 Apr 4, 2025
@martinlsm
Copy link
Collaborator Author

@pytorchbot label "topic: not user facing"

@martinlsm
Copy link
Collaborator Author

@pytorchbot label "partner: arm"

@pytorch-bot pytorch-bot bot added the partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm label Apr 4, 2025
@martinlsm
Copy link
Collaborator Author

@pytorchbot label ciflow/trunk

@pytorch pytorch deleted a comment from pytorch-bot bot Apr 4, 2025
@pytorch pytorch deleted a comment from pytorch-bot bot Apr 4, 2025
@zingo
Copy link
Collaborator

zingo commented Apr 4, 2025

qnn fails is unrelated

@zingo zingo merged commit bf3e0af into pytorch:main Apr 4, 2025
166 of 171 checks passed
@martinlsm martinlsm deleted the marlin-llama-skip branch April 6, 2025 21:58
kirklandsign pushed a commit that referenced this pull request Apr 11, 2025
If the user does not provide input files with the --llama_inputs
argument, the test is not run. Change the return statement into a
pytest.skip so that the test output show a skip rather than a pass.

Co-authored-by: Martin Lindström <[email protected]>
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.

4 participants