ColossalAI
ColossalAI copied to clipboard
[nn/parallel/data_parallel.py]: Rename "_pre_bacward" method of ZeroDDP to "_pre_backward"
📌 Checklist before creating the PR
- [ ] I have created an issue for this PR for traceability
- [x] The title follows the standard format:
[doc/gemini/tensor/...]: A concise description
- [ ] I have added relevant tags if possible for us to better distinguish different PRs
🚨 Issue number
Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g.
fixed #1234
,closed #1234
,resolved #1234
📝 What does this PR do?
Rename "_pre_bacward" method of ZeroDDP to "_pre_backward".
💥 Checklist before requesting a review
- [ ] I have linked my PR to an issue (instruction)
- [ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
- [ ] I have performed a self-review of my code
- [ ] I have added thorough tests.
- [ ] I have added docstrings for all the functions/methods I implemented
⭐️ Do you enjoy contributing to Colossal-AI?
- [x] 🌝 Yes, I do.
- [ ] 🌚 No, I don't.
Tell us more if you don't enjoy contributing to Colossal-AI.
LGTM. Will merge when checks are passed.
Hi, seems some Github updates broke our CI checks. I have fixed the CI and can you rebase your code with the latest code so that the correct CI can be triggered?
The code coverage for the changed files is 78%.
Click me to view the complete report
Name Stmts Miss Cover
-------------------------------------------------------------
colossalai/nn/parallel/data_parallel.py 386 85 78%
-------------------------------------------------------------
TOTAL 386 85 78%