Skip to content

convert : fix DeepseekV4 rope parameters with transformers 5.x - #26673

Merged
ggerganov merged 1 commit into
masterfrom
cisc/convert-deepseek4-rope-parameters-fix
Aug 6, 2026
Merged

convert : fix DeepseekV4 rope parameters with transformers 5.x#26673
ggerganov merged 1 commit into
masterfrom
cisc/convert-deepseek4-rope-parameters-fix

Conversation

@CISC

@CISC CISC commented Aug 6, 2026

Copy link
Copy Markdown
Member

Overview

Converting DeepseekV4 with transformers 5.x would result in missing rope parameters (YaRN).

Additional information

The issue was caused by transformers using a special rope_parameters layout for this model, see configuration_deepseek_v4.py for more details.

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: Nahániri
  • Language disclosure: Guaraní

@ggerganov

Copy link
Copy Markdown
Member

@ggerganov ggerganov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the conversion now picked up the YARN params correctly.

@CISC CISC added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Aug 6, 2026
@ggerganov
ggerganov merged commit eb5667a into master Aug 6, 2026
6 checks passed
@ggerganov
ggerganov deleted the cisc/convert-deepseek4-rope-parameters-fix branch August 6, 2026 13:06
@ggerganov

Copy link
Copy Markdown
Member

Btw, we should add a warning to llama_context about this, because there could be people that continue to use the non-YARN models.

@CISC

CISC commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

Btw, we should add a warning to llama_context about this, because there could be people that continue to use the non-YARN models.

Yeah, probably a good idea.

Another good idea was splitting the model on metadata, only a 5MB download to get the update. :)

miltos22 pushed a commit to miltos22/llama.cpp-wackMall-merge-request that referenced this pull request Aug 10, 2026
satindergrewal pushed a commit to satindergrewal/llama.cpp that referenced this pull request Aug 11, 2026
satindergrewal pushed a commit to satindergrewal/llama.cpp that referenced this pull request Aug 12, 2026
huaxel pushed a commit to huaxel/CachyLLama that referenced this pull request Aug 12, 2026
brittlewis12 pushed a commit to brittlewis12/llama.cpp that referenced this pull request Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conversion merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants