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

[RFC] Allow ModelWrapper and OptimizerWrapper to accept multiple models #360

Merged
merged 7 commits into from
Jun 5, 2024

Conversation

fegin
Copy link
Contributor

@fegin fegin commented May 24, 2024

[ghstack-poisoned]
fegin added a commit that referenced this pull request May 24, 2024
and optimizers

ghstack-source-id: 39ee905efb5c05acb62a0676402a4698b86dc6a3
Pull Request resolved: #360
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label May 24, 2024
@fegin fegin marked this pull request as draft May 24, 2024 07:40
[ghstack-poisoned]
fegin added a commit that referenced this pull request May 24, 2024
and optimizers

ghstack-source-id: fa6dc2cafb5dfdfced96896c89711fb6747743c1
Pull Request resolved: #360
[ghstack-poisoned]
fegin added a commit that referenced this pull request May 24, 2024
and optimizers

ghstack-source-id: 190220813ece188728a3c776e6839a323009f719
Pull Request resolved: #360
[ghstack-poisoned]
fegin added a commit that referenced this pull request Jun 3, 2024
…ls and optimizers

ghstack-source-id: 1e995493a6fd0c98a7714262d42d6bc3c2e1fe86
Pull Request resolved: #360
[ghstack-poisoned]
@fegin fegin marked this pull request as ready for review June 3, 2024 20:17
fegin added a commit that referenced this pull request Jun 3, 2024
…ls and optimizers

ghstack-source-id: 749a302774287ac182d649005d1f8d34491ec0d9
Pull Request resolved: #360
[ghstack-poisoned]
@wconstab wconstab mentioned this pull request Jun 4, 2024
[ghstack-poisoned]
fegin added a commit that referenced this pull request Jun 5, 2024
and optimizers

ghstack-source-id: 190220813ece188728a3c776e6839a323009f719
Pull Request resolved: #360
@wconstab wconstab merged commit f501714 into gh/fegin/3/base Jun 5, 2024
4 checks passed
wconstab pushed a commit that referenced this pull request Jun 5, 2024
and optimizers

ghstack-source-id: 190220813ece188728a3c776e6839a323009f719
Pull Request resolved: #360
@wconstab wconstab deleted the gh/fegin/3/head branch June 5, 2024 23:59
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 Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants