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

[llava][11/N] Migrate sampler to extension/llm #4460

Closed

Conversation

larryliu0820
Copy link
Contributor

@larryliu0820 larryliu0820 commented Jul 30, 2024

Stack from ghstack (oldest at bottom):

Summary: Move sampler code to extension/llm so that it can be reused by
llava runner.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D60458803

Summary: Move sampler code to extension/llm so that it can be reused by
llava runner.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Jul 30, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 7b2ba62 with merge base 1e14333 (image):
💚 Looks good so far! There are no failures yet. 💚

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

larryliu0820 added a commit that referenced this pull request Jul 30, 2024
Summary: Move sampler code to extension/llm so that it can be reused by
llava runner.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 25afaa8b3d6141991b9f1f4618ecaf3c7041dcc2
Pull Request resolved: #4460
@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 Jul 30, 2024
@larryliu0820
Copy link
Contributor Author

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 1ec3444.

dltn added a commit to dltn/executorch that referenced this pull request Aug 22, 2024
facebook-github-bot pushed a commit that referenced this pull request Aug 23, 2024
Differential Revision: D61693469

Pull Request resolved: #4853
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.

3 participants