Skip to content

Use bf16 type on model sharing#635

Merged
pefontana merged 4 commits intomainfrom
bf16-p2p-transfer
Mar 20, 2026
Merged

Use bf16 type on model sharing#635
pefontana merged 4 commits intomainfrom
bf16-p2p-transfer

Conversation

@IAvecilla
Copy link
Copy Markdown
Contributor

No description provided.

@IAvecilla IAvecilla self-assigned this Mar 16, 2026
Copy link
Copy Markdown
Contributor

@ethernet8023 ethernet8023 left a comment

Choose a reason for hiding this comment

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

approved with an Ok from Jeff - we should see about making sure checkpoints save and load from this format as well, and maybe throw an error if we try and load a checkpoint at any step other than 0 that's in any format other than fp16. thoughts?

Copy link
Copy Markdown
Contributor

@ethernet8023 ethernet8023 left a comment

Choose a reason for hiding this comment

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

i can't follow the barrer logic but it looks the same as the other methods, so lgtm.

@pefontana pefontana added this pull request to the merge queue Mar 20, 2026
Merged via the queue into main with commit 0bdb13d Mar 20, 2026
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants