[GH-ISSUE #23] why need convert.py? #19

Closed
opened 2026-10-26 21:11:06 +00:00 by bsakaguchi · 4 comments
Owner

Originally created by @CSEEduanyu on GitHub (Dec 29, 2024).
Original GitHub issue: https://github.com/deepseek-ai/DeepSeek-V3/issues/23

Originally created by @CSEEduanyu on GitHub (Dec 29, 2024). Original GitHub issue: https://github.com/deepseek-ai/DeepSeek-V3/issues/23
Author
Owner

@GeeeekExplorer commented on GitHub (Dec 31, 2024):

convert.py is used to convert huggingface checkpoints into model parallel checkpoints required by the demo, so that each rank only needs to load the corresponding checkpoint when running the demo

<!-- gh-comment-id:2566225594 --> @GeeeekExplorer commented on GitHub (Dec 31, 2024): `convert.py` is used to convert huggingface checkpoints into model parallel checkpoints required by the demo, so that each rank only needs to load the corresponding checkpoint when running the demo
Author
Owner

@KKwanhee commented on GitHub (Feb 3, 2025):

Why was layer61 (corresponding to nextn_predict_layers) excluded in convert.py when generating the safetensor? Doesn't that mean layer61 won't be used during inference?

<!-- gh-comment-id:2630244549 --> @KKwanhee commented on GitHub (Feb 3, 2025): Why was layer61 (corresponding to `nextn_predict_layers`) excluded in `convert.py` when generating the safetensor? Doesn't that mean `layer61` won't be used during inference?
Author
Owner

@GeeeekExplorer commented on GitHub (Feb 5, 2025):

Why was layer61 (corresponding to nextn_predict_layers) excluded in convert.py when generating the safetensor? Doesn't that mean layer61 won't be used during inference?

Yes, The infer demo currently does not support MTP which uses layer 61.

<!-- gh-comment-id:2635531804 --> @GeeeekExplorer commented on GitHub (Feb 5, 2025): > Why was layer61 (corresponding to `nextn_predict_layers`) excluded in `convert.py` when generating the safetensor? Doesn't that mean `layer61` won't be used during inference? Yes, The infer demo currently does not support MTP which uses layer 61.
Author
Owner

@Huixxi commented on GitHub (Feb 20, 2025):

But --model-parallel 32 seems also do not supported?

Why was layer61 (corresponding to nextn_predict_layers) excluded in convert.py when generating the safetensor? Doesn't that mean layer61 won't be used during inference?

Yes, The infer demo currently does not support MTP which uses layer 61.

<!-- gh-comment-id:2671291565 --> @Huixxi commented on GitHub (Feb 20, 2025): But --model-parallel 32 seems also do not supported? > > Why was layer61 (corresponding to `nextn_predict_layers`) excluded in `convert.py` when generating the safetensor? Doesn't that mean `layer61` won't be used during inference? > > Yes, The infer demo currently does not support MTP which uses layer 61.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
deepseek-ai/DeepSeek-V3#19
No description provided.