-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[CI] Cleanup Dist Optim tests with shared helper funcs #6125
base: main
Are you sure you want to change the base?
Conversation
Tests passed locally. |
Tests passed locally. |
@ver217 Can we fix this bot? He seems confused LOL |
@duanjunwen Would love a review when you are not busy. Thanks! |
@botbw Fixed test bugs and rebased to hopefully resolve the flash attn interface issue |
📌 Checklist before creating the PR
[doc/gemini/tensor/...]: A concise description
pip install pre-commit && pre-commit install
🚨 Issue number
📝 What does this PR do?
Removes multiple redundant functions for ease of maintaince and code convention.
💥 Checklist before requesting a review
⭐️ Do you enjoy contributing to Colossal-AI?
Tell us more if you don't enjoy contributing to Colossal-AI.