Skip to content

Conversation

@jbschlosser
Copy link
Contributor

Summary:
X-link: https://github.com/facebookresearch/FBGEMM/pull/2248

The meta registration(s) were not correct and compile testing was incorrectly not testing compile.

  • Pass compile=True to tests (D89333698)
  • Fix meta registrations
  • Use torch.no_grad() to avoid "missing backwards" errors for some compile tests
  • Call a separate test func for the decode path (D89346059)

Reviewed By: henrylhtsang

Differential Revision: D89398632

…pile

Summary:
X-link: facebookresearch/FBGEMM#2248

The meta registration(s) were not correct and compile testing was incorrectly not testing compile.
* Pass `compile=True` to tests (D89333698)
* Fix meta registrations
* Use `torch.no_grad()` to avoid "missing backwards" errors for some compile tests
* Call a separate test func for the decode path (D89346059)

Reviewed By: henrylhtsang

Differential Revision: D89398632
@meta-cla meta-cla bot added the cla signed label Dec 18, 2025
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Dec 18, 2025

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

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.

1 participant