Skip to content

Fix phi-3-mini build #5513

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Fix phi-3-mini build #5513

wants to merge 1 commit into from

Conversation

dbort
Copy link
Contributor

@dbort dbort commented Sep 20, 2024

Fix the build errors I introduced in #5318

Test Plan:
Built the model using the instructions at https://github.com/pytorch/executorch/blob/main/examples/models/phi-3-mini/README.md

@dbort dbort requested a review from helunwencser September 20, 2024 17:26
Copy link

pytorch-bot bot commented Sep 20, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 896d4dc with merge base af098c3 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 20, 2024
@facebook-github-bot
Copy link
Contributor

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

Fix the build errors I introduced in pytorch#5318

Test Plan:
Build the model using the instructions at https://github.com/pytorch/executorch/blob/main/examples/models/phi-3-mini/README.md
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@dbort merged this pull request in 8618607.

@dbort dbort deleted the phi3-fix branch September 20, 2024 22:32
@dbort
Copy link
Contributor Author

dbort commented Sep 20, 2024

@pytorchbot cherry-pick --onto release/0.4 -c regression

pytorchbot pushed a commit that referenced this pull request Sep 20, 2024
Summary:
Fix the build errors I introduced in #5318

Pull Request resolved: #5513

Test Plan: Built the model using the instructions at https://github.com/pytorch/executorch/blob/main/examples/models/phi-3-mini/README.md

Reviewed By: helunwencser

Differential Revision: D63133231

Pulled By: dbort

fbshipit-source-id: 81d5402e76fd86c919a13a1b1b83687feca72ab7
(cherry picked from commit 8618607)
@pytorchbot
Copy link
Collaborator

Cherry picking #5513

The cherry pick PR is at #5522 and it is recommended to link a regression cherry pick PR with an issue. The following tracker issues are updated:

Details for Dev Infra team Raised by workflow job

jackzhxng pushed a commit that referenced this pull request Sep 22, 2024
Fix phi-3-mini build (#5513)

Summary:
Fix the build errors I introduced in #5318

Pull Request resolved: #5513

Test Plan: Built the model using the instructions at https://github.com/pytorch/executorch/blob/main/examples/models/phi-3-mini/README.md

Reviewed By: helunwencser

Differential Revision: D63133231

Pulled By: dbort

fbshipit-source-id: 81d5402e76fd86c919a13a1b1b83687feca72ab7
(cherry picked from commit 8618607)

Co-authored-by: Dave Bort <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants