Skip to content

convert : allow vision target for DFlash/DSpark - #29339

Merged
ngxson merged 1 commit into
ggml-org:masterfrom
tdakhran:tarek/feat/dspark-lfm25-vl
Sep 23, 2026
Merged

ngxson merged 1 commit into
ggml-org:masterfrom
tdakhran:tarek/feat/dspark-lfm25-vl

Conversation

@tdakhran

Copy link
Copy Markdown
Contributor

Overview

PR supersedes #29332.

Resolve the target architecture with get_model_architecture so vision targets (e.g. Lfm2VlForConditionalGeneration) map to their text model for the vocab.

Additional information

Fix double rope reordering for LFM2/LFM2.5 DSpark drafters.
The first time reordering was introduced in #27383 for LFM2 text models.
Then it was implemented again in #27342, leading to double reordering.

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: YES, for debugging double rope reordering

Resolve the target arch with get_model_architecture so vision targets
(e.g. Lfm2VlForConditionalGeneration) map to their text model for the vocab.

Fix double rope reorder for LFM2/LFM2.5 DSpark drafters
@tdakhran

Copy link
Copy Markdown
Contributor Author

cc: @tugot17 @ngxson

@CISC

CISC commented Sep 23, 2026

Copy link
Copy Markdown
Member

Fix double rope reordering for LFM2/LFM2.5 DSpark drafters. The first time reordering was introduced in #27383 for LFM2 text models. Then it was implemented again in #27342, leading to double reordering.

Well, that was silly. :D

@ngxson
ngxson merged commit bd4f514 into ggml-org:master Sep 23, 2026
5 checks passed
@ngxson

ngxson commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

thanks @tdakhran for the quick response! 🚀

@tdakhran
tdakhran deleted the tarek/feat/dspark-lfm25-vl branch September 24, 2026 00:00
turbo-tan pushed a commit to turbo-tan/llama.cpp-tq3 that referenced this pull request Sep 24, 2026
Resolve the target arch with get_model_architecture so vision targets
(e.g. Lfm2VlForConditionalGeneration) map to their text model for the vocab.

Fix double rope reorder for LFM2/LFM2.5 DSpark drafters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants