Skip to content

Simplify is_torchdynamo_compiling to direct import from torch.compiler#5618

Closed
cyyever wants to merge 1 commit intopytorch:mainfrom
cyyever:remove-is-torchdynamo-compiling
Closed

Simplify is_torchdynamo_compiling to direct import from torch.compiler#5618
cyyever wants to merge 1 commit intopytorch:mainfrom
cyyever:remove-is-torchdynamo-compiling

Conversation

@cyyever
Copy link
Copy Markdown
Contributor

@cyyever cyyever commented Apr 11, 2026

No description provided.

@meta-cla meta-cla bot added the cla signed label Apr 11, 2026
@q10
Copy link
Copy Markdown
Contributor

q10 commented Apr 11, 2026

I think we might want to keep the existing code for backward compatibility reasons; there's a good chance it might break internal published models.

@cyyever cyyever marked this pull request as draft April 12, 2026 00:01
@cyyever cyyever marked this pull request as ready for review April 12, 2026 02:22
@cyyever
Copy link
Copy Markdown
Contributor Author

cyyever commented Apr 12, 2026

@q10 it should work if the internal modules all use PyTorch 2.

@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Apr 13, 2026

@q10 has imported this pull request. If you are a Meta employee, you can view this in D100559702.

@cyyever cyyever force-pushed the remove-is-torchdynamo-compiling branch from 32a96ca to 1e3edb0 Compare April 13, 2026 07:19
q10 pushed a commit to q10/FBGEMM that referenced this pull request Apr 14, 2026
pytorch#5618)

Summary:
X-link: facebookresearch/FBGEMM#2575




Reviewed By: henrylhtsang

Differential Revision: D100559702

Pulled By: q10
@meta-codesync meta-codesync bot closed this in 2e3da8a Apr 14, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Apr 14, 2026

@q10 merged this pull request in 2e3da8a.

@cyyever cyyever deleted the remove-is-torchdynamo-compiling branch April 15, 2026 01:51
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