Skip to content
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

Fix missing move and bare new in pytree from_str_internal #6783

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

swolchok
Copy link
Contributor

@swolchok swolchok commented Nov 12, 2024

Stack from ghstack (oldest at bottom):

Just a couple minor fixes.

Differential Revision: D65576543

Just a couple minor fixes.

Differential Revision: [D65576543](https://our.internmc.facebook.com/intern/diff/D65576543/)

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Nov 12, 2024

🔗 Helpful Links

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

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

❌ 1 New Failure

As of commit b2d6a4c with merge base dc41596 (image):

NEW FAILURE - The following job has failed:

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 Nov 12, 2024
swolchok added a commit that referenced this pull request Nov 12, 2024
Just a couple minor fixes.

Differential Revision: [D65576543](https://our.internmc.facebook.com/intern/diff/D65576543/)

ghstack-source-id: 253128072
Pull Request resolved: #6783
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D65576543

@swolchok
Copy link
Contributor Author

Had to re-export this one because I landed it wrong when it was #6702; apparently I am supposed to click the internal land button.

@swolchok
Copy link
Contributor Author

extension/llm/modules/test/test_mha.py::AttentionTest::test_attention_executorch seems to be flaky -- https://hud.pytorch.org/hud/pytorch/executorch/main/1?per_page=50&name_filter=unittest&mergeLF=true . bypassing

@swolchok swolchok merged commit 6d7a1d8 into gh/swolchok/115/base Nov 12, 2024
39 of 41 checks passed
@swolchok swolchok deleted the gh/swolchok/115/head branch November 12, 2024 23:28
swolchok added a commit that referenced this pull request Nov 12, 2024
* Use std::variant to implement pytree Key

Pull Request resolved: #6701

Key was a struct that should've been a union; std::variant makes using a union much easier.
ghstack-source-id: 253128071
@exported-using-ghexport

Differential Revision: [D65575184](https://our.internmc.facebook.com/intern/diff/D65575184/)

* Fix missing move and bare new in pytree from_str_internal

Just a couple minor fixes.

Differential Revision: [D65576543](https://our.internmc.facebook.com/intern/diff/D65576543/)

ghstack-source-id: 253128072
Pull Request resolved: #6783

---------

Co-authored-by: Scott Wolchok <[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. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants