Skip to content

Conversation

@SaoirseARM
Copy link
Collaborator

@SaoirseARM SaoirseARM commented Jan 21, 2025

Summary

  • Adding LSTM unit test for MI, BI, U55 and U85
  • Includes just one sample input.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

@SaoirseARM SaoirseARM requested a review from zingo January 21, 2025 16:43
@pytorch-bot
Copy link

pytorch-bot bot commented Jan 21, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 3 Cancelled Jobs

As of commit 6cb41ed with merge base cd9ab87 (image):

NEW FAILURES - The following jobs have failed:

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 Jan 21, 2025
@SaoirseARM
Copy link
Collaborator Author

@pytorchbot label ciflow/trunk

@SaoirseARM
Copy link
Collaborator Author

@pytorchbot label "topic: not user facing"

@SaoirseARM
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 Jan 21, 2025
Copy link
Contributor

@digantdesai digantdesai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stamping to unblock merging.

- Adding check that number of delegates=1
@zingo
Copy link
Collaborator

zingo commented Jan 23, 2025

Fails seems unrelated to PR and fails on other PR's also.

@zingo zingo merged commit 2516049 into pytorch:main Jan 23, 2025
103 of 108 checks passed
YIWENX14 pushed a commit that referenced this pull request Jan 28, 2025
Add LSTM unit test to arm backend
Adding check that number of delegates=1
zonglinpeng pushed a commit to zonglinpeng/executorch that referenced this pull request Jan 30, 2025
Add LSTM unit test to arm backend
Adding check that number of delegates=1
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