Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add 8bit quant all reduce support #11

Merged
merged 1 commit into from
Aug 2, 2024
Merged

add 8bit quant all reduce support #11

merged 1 commit into from
Aug 2, 2024

Conversation

samsja
Copy link
Member

@samsja samsja commented Aug 1, 2024

No description provided.

@samsja samsja requested a review from Jackmin801 August 1, 2024 15:38
@samsja samsja force-pushed the feat-allow-8bit-grad branch from d15ac9e to 8e066fe Compare August 1, 2024 15:41
Comment on lines -112 to -114
raise ValueError(
f"Invalid hivemind_compression: {hivemind_compression}. Please choose 'none', 'fp16', or 'scaled-fp16'."
)
Copy link
Member Author

Choose a reason for hiding this comment

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

the config is already catching mis naming and propose suggestion thanks to literal support in pydantic

Copy link
Member

Choose a reason for hiding this comment

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

ah ok great!

@samsja samsja force-pushed the feat-allow-8bit-grad branch from 8e066fe to 3f3c27d Compare August 1, 2024 16:25
Copy link
Member

@Jackmin801 Jackmin801 left a comment

Choose a reason for hiding this comment

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

lgtm!

Comment on lines -112 to -114
raise ValueError(
f"Invalid hivemind_compression: {hivemind_compression}. Please choose 'none', 'fp16', or 'scaled-fp16'."
)
Copy link
Member

Choose a reason for hiding this comment

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

ah ok great!

@samsja samsja merged commit 32395eb into main Aug 2, 2024
2 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.

2 participants