Skip to content

Add unit test to validate HIP backward kernel with FP16 momentum (#5519)#5519

Closed
q10 wants to merge 1 commit into
pytorch:mainfrom
q10:export-D97697077
Closed

Add unit test to validate HIP backward kernel with FP16 momentum (#5519)#5519
q10 wants to merge 1 commit into
pytorch:mainfrom
q10:export-D97697077

Conversation

@q10

@q10 q10 commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

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

  • Add unit test to validate HIP backward kernel with FP16 momentum

Reviewed By: cthi

Differential Revision: D97697077

@meta-cla meta-cla Bot added the cla signed label Mar 24, 2026
@meta-codesync

meta-codesync Bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

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

@meta-codesync meta-codesync Bot changed the title Add unit test to validate HIP backward kernel with FP16 momentum Add unit test to validate HIP backward kernel with FP16 momentum (#5519) Mar 24, 2026
@q10 q10 force-pushed the export-D97697077 branch from 860a6d9 to 03627e6 Compare March 24, 2026 08:03
q10 added a commit to q10/FBGEMM that referenced this pull request Mar 24, 2026
…orch#5519)

Summary:
X-link: facebookresearch/FBGEMM#2492


- Add unit test to validate HIP backward kernel with FP16 momentum

Reviewed By: cthi

Differential Revision: D97697077
q10 added a commit to q10/FBGEMM that referenced this pull request Mar 24, 2026
…orch#5519)

Summary:
X-link: facebookresearch/FBGEMM#2492


- Add unit test to validate HIP backward kernel with FP16 momentum

Reviewed By: cthi

Differential Revision: D97697077
@q10 q10 force-pushed the export-D97697077 branch from 03627e6 to 8b9c053 Compare March 24, 2026 08:03
…orch#5519)

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

Pull Request resolved: pytorch#5519

- Add unit test to validate HIP backward kernel with FP16 momentum

Reviewed By: cthi

Differential Revision: D97697077
@meta-codesync

meta-codesync Bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

This pull request has been merged in 52d44ac.

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