Skip to content

[export] Support unflatten of graph mixing placeholder and get_attr#164077

Closed
kwen2501 wants to merge 2 commits intogh/kwen2501/268/basefrom
gh/kwen2501/268/head
Closed

[export] Support unflatten of graph mixing placeholder and get_attr#164077
kwen2501 wants to merge 2 commits intogh/kwen2501/268/basefrom
gh/kwen2501/268/head

Conversation

@kwen2501
Copy link
Collaborator

@kwen2501 kwen2501 commented Sep 28, 2025

Stack from ghstack (oldest at bottom):

Fixes #164083.

Pre-process all placeholder nodes, record the first non-placeholder, and run_from there.
In run_from, ignore any placeholder, instead of issuing a hard assert.

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 28, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 57bd493 with merge base 8d81564 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[ghstack-poisoned]
Khanaksahu pushed a commit to Khanaksahu/pytorch-fork that referenced this pull request Nov 17, 2025
@github-actions
Copy link
Contributor

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Jan 21, 2026
@github-actions github-actions bot closed this Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants