Skip to content

Conversation

SaoirseARM
Copy link
Collaborator

@SaoirseARM SaoirseARM commented Mar 21, 2025

Summary

  • Adds new test pipeline which runs transform_for_annotation_pipeline on initial exported graph.
  • Compares the Initial model against exported model in eager mode to check numerical differences.
  • This helps to discover issues in the passes before quantization.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

@SaoirseARM SaoirseARM requested a review from digantdesai as a code owner March 21, 2025 10:15
Copy link

pytorch-bot bot commented Mar 21, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Unrelated Failure

As of commit 5098e83 with merge base 95d7cce (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

@pytorchbot label ciflow/trunk

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

@pytorchbot label "topic: not user facing"

Copy link
Collaborator

@oscarandersson8218 oscarandersson8218 left a comment

Choose a reason for hiding this comment

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

Looks good to me. CI failures seem unrelated.

@oscarandersson8218 oscarandersson8218 merged commit e22b21a into pytorch:main Mar 21, 2025
161 of 164 checks passed
DannyYuyang-quic pushed a commit to CodeLinaro/executorch that referenced this pull request Apr 2, 2025
…ine (pytorch#9488)

### Summary
- Adds new test pipeline which runs transform_for_annotation_pipeline on
initial exported graph.
- Compares the Initial model against exported model in eager mode to
check numerical differences.
- This helps to discover issues in the passes before quantization.

Signed-off-by: Saoirse Stewart <[email protected]>
Change-Id: I0ecbf5167857f8e3c8f9426fffa4e5fc578003af
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