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

sync: main to dev #144

Merged
merged 6 commits into from
Apr 17, 2024
Merged

sync: main to dev #144

merged 6 commits into from
Apr 17, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Apr 8, 2024

sync-branches: New code has just landed in main, so let's bring dev up to speed!

HeyyyyyyG and others added 2 commits April 7, 2024 18:04
* initial commit for lora dpo

Signed-off-by: Jiaqi Zeng <[email protected]>

* Lora for SFT/SteerLM (#97)

* initial commit: peft for sft

Signed-off-by: Jiaqi Zeng <[email protected]>

* lora only

Signed-off-by: Jiaqi Zeng <[email protected]>

* formatting

Signed-off-by: Jiaqi Zeng <[email protected]>

* minor change

Signed-off-by: Jiaqi Zeng <[email protected]>

* add assertion

Signed-off-by: Jiaqi Zeng <[email protected]>

* default to full sft

Signed-off-by: Jiaqi Zeng <[email protected]>

* enable O2 for lora

Signed-off-by: Jiaqi Zeng <[email protected]>

* update CHANGELOG.md

Signed-off-by: Jiaqi Zeng <[email protected]>

* move peft initialization to train_script_utils.py

Signed-off-by: Jiaqi Zeng <[email protected]>

* remove check for distributed_fused_adam & move setup_complete to init_peft

Signed-off-by: Jiaqi Zeng <[email protected]>

* update Dockerfile

Signed-off-by: Jiaqi Zeng <[email protected]>

* fix cherry-pick commit id

Signed-off-by: Jiaqi Zeng <[email protected]>

---------

Signed-off-by: Jiaqi Zeng <[email protected]>

* load_from_nemo bug fix (#103)

Signed-off-by: Jiaqi Zeng <[email protected]>

* use init_peft

Signed-off-by: Jiaqi Zeng <[email protected]>

* change default to full tuning

Signed-off-by: Jiaqi Zeng <[email protected]>

* remove duplicates

Signed-off-by: Jiaqi Zeng <[email protected]>

* update CHANGELOG.md

Signed-off-by: Jiaqi Zeng <[email protected]>

* address comments

Signed-off-by: Jiaqi Zeng <[email protected]>

* update documentation

Signed-off-by: Jiaqi Zeng <[email protected]>

---------

Signed-off-by: Jiaqi Zeng <[email protected]>
@github-actions github-actions bot added documentation Improvements or additions to documentation Utils labels Apr 9, 2024
odelalleau and others added 4 commits April 9, 2024 16:45
Signed-off-by: Gerald Shen <[email protected]>
Co-authored-by: Olivier Delalleau <[email protected]>
* fix confusion on checkpoint to use

Signed-off-by: Gerald Shen <[email protected]>

* fix typo

Signed-off-by: Gerald Shen <[email protected]>

---------

Signed-off-by: Gerald Shen <[email protected]>
Signed-off-by: Gerald Shen <[email protected]>
@gshennvm gshennvm merged commit 2cf59b5 into dev Apr 17, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Utils
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants