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

1-bit adam #353

Merged
merged 22 commits into from
Sep 8, 2020
Merged

1-bit adam #353

merged 22 commits into from
Sep 8, 2020

Conversation

awan-10
Copy link
Contributor

@awan-10 awan-10 commented Sep 3, 2020

No description provided.

jeffra and others added 2 commits September 2, 2020 11:47
* testing_onebit

* test_passed

* test_passed

* updated compressed_allreduce

* 123

* cpu2gpu test added

* Add non-cuda-aware code path. Segfaults for > 2 procs.

* Works for 4 procs with numpy buffers now. TODO: cleanup, evalute perf.

* Fix gather. Cleanup.

* Add new tests.

* Reduce memory footprint. BERT large with BS=16 works.

* Revert "Reduce memory footprint. BERT large with BS=16 works."

This reverts commit e7f38fc.

* Update optim to support bert-large.

* with initialization added on bert_onebit_adam

* This works!!

* Force igather for cupy. Better performance. Need cleanup and reorg to
support TCP now.

* X

* testing the fintune task for FP32 training

* Added the fintune taks for FP32 training

* With the control flag of freeze_step added

* added the freeze_step inside fp32_onebitadam

* Seperate freeze_kernnel added

* Added the sanity test for the Compressed_Allreduce

* Test for Compressed_Allreduce passed, but AllGather need sync.

* add checks for finetuning.

* Running passed for finetune on EastUs

* Add one bit adam clean file.

* Refactor comms. code and move it to a new file.

* fix compile/run errors.

* Adding changes for onebit_adam from Hank

* Save memory by modifying in-place.

Co-authored-by: Your Name <[email protected]>
Co-authored-by: Ammar Ahmad Awan <[email protected]>
Co-authored-by: tanghl1994 <[email protected]>
Co-authored-by: Hank <[email protected]>
Co-authored-by: root <[email protected]>
* Refactor to correct locations.

* Deleted unused files.

* Fix imports for refactored codebase.

* update the com reduce test.

* Fix some errors.

* Fix optimizer name

* Delete unused tests.

* Fix formatting for pre-commit.

* Add cupy dependencies.

* add cupy for cuda 10add cupy for cuda 10

* Add mpi4py requirement.

Co-authored-by: Ammar Ahmad Awan <[email protected]>
@ShadenSmith ShadenSmith marked this pull request as draft September 6, 2020 15:16
@ShadenSmith
Copy link
Contributor

Marked PR as a draft until we are ready to merge.

deepspeed/runtime/utils.py Outdated Show resolved Hide resolved
@jeffra jeffra marked this pull request as ready for review September 8, 2020 05:02
@jeffra jeffra merged commit fa66867 into staging-1bit-adam-v2 Sep 8, 2020
jeffra added a commit that referenced this pull request Sep 9, 2020
* 1-bit adam (#353)

Co-authored-by: Jeff Rasley <[email protected]>
Co-authored-by: Your Name <[email protected]>
Co-authored-by: tanghl1994 <[email protected]>
Co-authored-by: Hank <[email protected]>
Co-authored-by: root <[email protected]>
Co-authored-by: Ammar Ahmad Awan <[email protected]>
@jeffra jeffra deleted the staging-1bit-adam-v1 branch November 19, 2020 23:27
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.

4 participants