Skip to content

Attempt to fix FBGEMM CPU build#3499

Closed
huydhn wants to merge 2 commits intomainfrom
fix-cpu-build-manywheel-gcc
Closed

Attempt to fix FBGEMM CPU build#3499
huydhn wants to merge 2 commits intomainfrom
fix-cpu-build-manywheel-gcc

Conversation

@huydhn
Copy link
Copy Markdown
Contributor

@huydhn huydhn commented Dec 11, 2024

Attempt to fix https://github.com/pytorch/FBGEMM/actions/runs/12209285300/job/34063862021 before we have the new manywheel 2.28 container with the new toolchain available.

Testing

Looks ok now https://github.com/pytorch/FBGEMM/actions/runs/12283462291

@netlify
Copy link
Copy Markdown

netlify bot commented Dec 11, 2024

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit 14e57c4
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/6759e707e379ba0008189ec2
😎 Deploy Preview https://deploy-preview-3499--pytorch-fbgemm-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@huydhn huydhn requested a review from q10 December 11, 2024 21:23
@huydhn huydhn marked this pull request as ready for review December 11, 2024 21:23
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@huydhn has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@huydhn merged this pull request in 5c16f4b.

@huydhn huydhn deleted the fix-cpu-build-manywheel-gcc branch December 23, 2024 17:25
huydhn added a commit that referenced this pull request Dec 23, 2024
facebook-github-bot pushed a commit that referenced this pull request Dec 26, 2024
Summary:
X-link: facebookresearch/FBGEMM#612

This reverts commit 5c16f4b.  This is not needed anymore after pytorch/pytorch#143423.  I think this will also fix the issue with building torchrec CPU https://github.com/pytorch/FBGEMM/actions/runs/12470608879/job/34806045264?pr=3528#step:18:219⁩

### Testing

https://github.com/pytorch/FBGEMM/actions/runs/12470608879

Pull Request resolved: #3528

Reviewed By: q10

Differential Revision: D67602736

Pulled By: huydhn

fbshipit-source-id: ec5888acacd96295dd6dfe26e5fa87b28810b2bc
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.

4 participants