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

chore(release): merge set of changes for v2.4.0 #439

Closed
wants to merge 8 commits into from
Closed

Conversation

willmj
Copy link
Collaborator

@willmj willmj commented Jan 16, 2025

release 2.4.0!

dushyantbehl and others added 8 commits December 24, 2024 00:10
Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>
fix: broken README.md link
* Allow hf dataset id to be loaded by training_data_path

Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>

* update README

Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>

* minor changes

Signed-off-by: Abhishek <maurya.abhishek@ibm.com>

---------

Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>
Signed-off-by: Abhishek <maurya.abhishek@ibm.com>
Co-authored-by: Abhishek <maurya.abhishek@ibm.com>
* feat: accelerated MoE dataclass and init

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: author's note

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* feat: accelerated moe in acceleration framework

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* feat: accelerated moe to sft_trainer

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* feat: fmt, testing

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: rename accelerated moe to fast moe

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* test: add testing for scatter moe on accel framework

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: model, dtype, assertions

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: post init check removed from FastMoe, experimental set to True

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: if non-iterable nested dataclass, still initialize

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* test: add failing test for wrong ep_degree

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: actually expect failure

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* test: make sure fast moe doesn't work with non-moe model

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: regex of new test

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* comment: explain iterable unpacking

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* docs: fast MOE in README

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* docs: Add note for post-processing

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: Dockerfile

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* test: fix params

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: file path

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: expand on docs, remove from Dockerfile, move iterable data to else statement

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* lint

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: spelling

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

---------

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>
…ate (#430)

* fix: allow for padding free + pretraining

Signed-off-by: Harikrishnan Balagopal <harikrishmenon@gmail.com>

* add data collator for padding free plugin scenario to be used for extended pretraining

Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>

* fix: update value error

Signed-off-by: Mehant Kammakomati <mehant.kammakomati2@ibm.com>

* fix: delete images only when exists

Signed-off-by: Mehant Kammakomati <mehant.kammakomati2@ibm.com>

---------

Signed-off-by: Harikrishnan Balagopal <harikrishmenon@gmail.com>
Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>
Signed-off-by: Mehant Kammakomati <mehant.kammakomati2@ibm.com>
Co-authored-by: Harikrishnan Balagopal <harikrishmenon@gmail.com>
Co-authored-by: Mehant Kammakomati <mehant.kammakomati2@ibm.com>
#434)

* fix: function name from requires_agumentation to requires_augmentation

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fix: file path

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

* fmt

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>

---------

Signed-off-by: Will Johnson <mwjohnson728@gmail.com>
Signed-off-by: Abhishek <maurya.abhishek@ibm.com>
Signed-off-by: Will Johnson <mwjohnson728@gmail.com>
Copy link

Thanks for making a pull request! 😃
One of the maintainers will review and advise on the next steps.

@github-actions github-actions bot added the chore label Jan 16, 2025
@aluu317
Copy link
Collaborator

aluu317 commented Jan 16, 2025

This looks good, but could we wait 1 day to merge? Waiting to test something for PR #422 and see if I can get it merged soon

Copy link
Collaborator

@Abhishek-TAMU Abhishek-TAMU left a comment

Choose a reason for hiding this comment

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

The current commits are in sync with the main branch. Looks good to me.

@willmj willmj closed this Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants