Skip to content

Conversation

@sxu
Copy link
Contributor

@sxu sxu commented Oct 15, 2025

Summary: Introduce a source transform to be more aligned with other transforms we run, also makes it less error prone (e.g. HF RoPE transformation needs to happen before turning linears into conv2ds).

Differential Revision: D84769599

@sxu sxu requested review from jackzhxng and lucylq as code owners October 15, 2025 23:42
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 15, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Unrelated Failure

As of commit 1bc10dd with merge base e1d9fd2 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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 15, 2025
@meta-codesync
Copy link

meta-codesync bot commented Oct 15, 2025

@sxu has exported this pull request. If you are a Meta employee, you can view the originating Diff in D84769599.

@sxu sxu added the release notes: none Do not include this in the release notes label Oct 15, 2025
Summary:

Introduce a source transform to be more aligned with other transforms we run, also makes it less error prone (e.g. HF RoPE transformation needs to happen before turning linears into conv2ds).

Differential Revision: D84769599
@sxu sxu force-pushed the export-D84769599 branch from f19b4e9 to 1bc10dd Compare October 15, 2025 23:59
@meta-codesync meta-codesync bot merged commit 9287f6d into pytorch:main Oct 16, 2025
136 of 141 checks passed
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. fb-exported meta-exported 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