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

Hsdp + MoE CI tests #3378

Merged
merged 72 commits into from
Jun 24, 2024
Merged

Hsdp + MoE CI tests #3378

merged 72 commits into from
Jun 24, 2024

Conversation

KuuCi
Copy link
Contributor

@KuuCi KuuCi commented Jun 6, 2024

What does this PR do?

This PR is meant to add HSDP and MoE CI tests to composer using 4GPUs.

What issue(s) does this change relate to?

Before submitting

  • Have you read the contributor guidelines?
  • Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
  • Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
  • Did you update any related docs and document your change?
  • Did you update any related tests and add any new tests related to your change? (see testing)
  • Did you run the tests locally to make sure they pass?
  • Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

@KuuCi KuuCi requested a review from mvpatel2000 June 20, 2024 23:29
@KuuCi KuuCi marked this pull request as ready for review June 20, 2024 23:43
Copy link
Contributor

@mvpatel2000 mvpatel2000 left a comment

Choose a reason for hiding this comment

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

can you remove world_size in param as you are now explicitly parameterizing each test

tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
tests/trainer/test_fsdp_checkpoint.py Show resolved Hide resolved
tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
@KuuCi KuuCi requested a review from mvpatel2000 June 21, 2024 22:17
@KuuCi KuuCi requested review from bigning June 24, 2024 17:53
tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
tests/trainer/test_fsdp_checkpoint.py Outdated Show resolved Hide resolved
KuuCi and others added 2 commits June 24, 2024 17:07
@KuuCi KuuCi merged commit 072758e into dev Jun 24, 2024
17 checks passed
@KuuCi KuuCi deleted the hsdp-ci-tests branch June 24, 2024 22:30
mvpatel2000 added a commit to mvpatel2000/composer that referenced this pull request Jul 21, 2024
* fold ema fsdp state

* debug

* debug

* more debug

* keep debugging

* debug

* sanity check

* debug

* debug

* use ema

* debug

* debug

* debug

* debug

* debug

* debug

* more fix

* filename test

* revert test

* fully parameterize

* hsdp test

* revert testing

* typo

* typo

* hsdp

* split off test

* precommit

* float to int

* pyright

* oom

* print

* rm tp

* tp cfg

* tp?

* rm tp line

* type annotation

* revert

* readd tp

* type

* world size

* revert

* revert monolithic cpkt + include sharded cpkt

* enumerate

* precommit

* precommit

* sharded

* sync

* only sync on first trainer

* typo

* hsdp

* xfail

* explicit sync

* test

* revert test

* sync, docker issue

* pre-commit

* sync

* pytest

* xfail

* rm world_size param

* im so sorry pls forgive me king

* the kings comments

* Update tests/trainer/test_fsdp_checkpoint.py

fix formatting

Co-authored-by: Mihir Patel <[email protected]>

* precommit

---------

Co-authored-by: v-chen_data <[email protected]>
Co-authored-by: Daniel King <[email protected]>
Co-authored-by: Mihir Patel <[email protected]>
mvpatel2000 added a commit that referenced this pull request Jul 21, 2024
* fold ema fsdp state

* debug

* debug

* more debug

* keep debugging

* debug

* sanity check

* debug

* debug

* use ema

* debug

* debug

* debug

* debug

* debug

* debug

* more fix

* filename test

* revert test

* fully parameterize

* hsdp test

* revert testing

* typo

* typo

* hsdp

* split off test

* precommit

* float to int

* pyright

* oom

* print

* rm tp

* tp cfg

* tp?

* rm tp line

* type annotation

* revert

* readd tp

* type

* world size

* revert

* revert monolithic cpkt + include sharded cpkt

* enumerate

* precommit

* precommit

* sharded

* sync

* only sync on first trainer

* typo

* hsdp

* xfail

* explicit sync

* test

* revert test

* sync, docker issue

* pre-commit

* sync

* pytest

* xfail

* rm world_size param

* im so sorry pls forgive me king

* the kings comments

* Update tests/trainer/test_fsdp_checkpoint.py

fix formatting

Co-authored-by: Mihir Patel <[email protected]>

* precommit

---------

Co-authored-by: v-chen_data <[email protected]>
Co-authored-by: Daniel King <[email protected]>
Co-authored-by: Mihir Patel <[email protected]>
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