vllm icon indicating copy to clipboard operation
vllm copied to clipboard

[Feature]: add DoRA support

Open cmhungsteve opened this issue 1 year ago • 7 comments

🚀 The feature, motivation and pitch

The recent ICML'24 Oral paper, DoRA, has shown consistent improvement over LoRA on various tasks (LLM, MLLM, etc.) and backbones (LLaMA, LLaVA, etc.) [Code] https://github.com/NVlabs/DoRA

DoRA has also been integrated into various open-source library/framework:

teaser_v2_bg

Alternatives

No response

Additional context

No response

Before submitting a new issue...

  • [X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

cmhungsteve avatar Dec 03 '24 04:12 cmhungsteve

Thank you very much for bringing this feature. We will consider supporting it.

jeejeelee avatar Dec 03 '24 05:12 jeejeelee

I also need vllm to support dora !!!

TC10127 avatar Dec 03 '24 06:12 TC10127

We need this feature too,this will help a lot!

PacinoAn avatar Dec 03 '24 12:12 PacinoAn

@jeejeelee any updates on whether this is on the roadmap?

winglian avatar Feb 03 '25 02:02 winglian

@jeejeelee https://x.com/winglian/status/1888951180606202028 GRPO + DoRA converges faster than GRPO+ FFT or GRPO + LoRA (thanks @winglian for the great finding!)

cmhungsteve avatar Feb 13 '25 06:02 cmhungsteve

@jeejeelee https://x.com/winglian/status/1888951180606202028 GRPO + DoRA converges faster than GRPO+ FFT or GRPO + LoRA (thanks @winglian for the great finding!)

Thanks, will start trying to support DoRA soon

jeejeelee avatar Feb 13 '25 07:02 jeejeelee

Hi, this is my first time contributing to vllm, and to opensource in general, but I've created this PR for adding DoRA support: https://github.com/vllm-project/vllm/pull/14389

Thanks for your patience with my opensource development practices, and I look forward to learning with you!

ChloeL19 avatar Mar 07 '25 00:03 ChloeL19

This issue has been automatically marked as stale because it has not had any activity within 90 days. It will be automatically closed if no further activity occurs within 30 days. Leave a comment if you feel this issue should remain open. Thank you!

github-actions[bot] avatar Jun 05 '25 02:06 github-actions[bot]

This issue has been automatically closed due to inactivity. Please feel free to reopen if you feel it is still relevant. Thank you!

github-actions[bot] avatar Jul 06 '25 02:07 github-actions[bot]

@jeejeelee just curious: any plan or timeline for the support?

cmhungsteve avatar Jul 06 '25 07:07 cmhungsteve

I have considered this issue before. A rather tricky problem is that the TP>1 case is not easy to handle

jeejeelee avatar Jul 25 '25 16:07 jeejeelee

@jeejeelee should I mark this as keep-open so the bot leaves it alone?

hmellor avatar Aug 28 '25 09:08 hmellor

Let's keep it open, thank you

jeejeelee avatar Aug 28 '25 11:08 jeejeelee

Hello, do you know if Dora is supported at this moment ? I still have : ValueError: vLLM does not yet support DoRA. on vllm=0.11.0

HayatoSempai avatar Oct 13 '25 07:10 HayatoSempai