Skip to content

add speculators finetune examples#339

Closed
Annarine wants to merge 1 commit into
vllm-project:mainfrom
Annarine:finetune-test
Closed

add speculators finetune examples#339
Annarine wants to merge 1 commit into
vllm-project:mainfrom
Annarine:finetune-test

Conversation

@Annarine

@Annarine Annarine commented Mar 10, 2026

Copy link
Copy Markdown

PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.

Purpose

Description

这个PR需要依赖#325

Related Issue

Tests

I have filled in:

  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan/results, such as providing test command and pasting the results.
  • (Optional) The necessary documentation update.
  • I (a human) have written or reviewed the code in this pr to the best of my ability.

@mergify mergify Bot added the documentation Improvements or additions to documentation label Mar 10, 2026
@momo609

momo609 commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

please add related pr.

@fynnsu

fynnsu commented Mar 11, 2026

Copy link
Copy Markdown
Collaborator

Finetuning support was merged in #333, instead of #325. Could you update this pr to work with that? I think the cli argument name is slightly different. Otherwise should be mostly the same.

@mergify

mergify Bot commented Mar 17, 2026

Copy link
Copy Markdown

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @Annarine.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase label Mar 17, 2026
@mergify mergify Bot removed the needs-rebase label Apr 22, 2026
@mergify

mergify Bot commented Apr 22, 2026

Copy link
Copy Markdown

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @Annarine.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase label Apr 22, 2026
@fynnsu

fynnsu commented May 21, 2026

Copy link
Copy Markdown
Collaborator

Closing as this is outdated and uses the old gen_and_train script which has been removed.

@fynnsu fynnsu closed this May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation needs-rebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants