Skip to content

Add SM margin to LayerNorm in inference - #772

Merged
ksivaman merged 5 commits into
NVIDIA:mainfrom
erhoo82:ln_margin
Apr 12, 2024
Merged

Add SM margin to LayerNorm in inference#772
ksivaman merged 5 commits into
NVIDIA:mainfrom
erhoo82:ln_margin

Conversation

@erhoo82

Copy link
Copy Markdown
Collaborator

Add SM margin to LayerNorm in inference to enable the overlap of parameter AllGather with the validation step

Signed-off-by: Sangkug Lym <slym@nvidia.com>
@ptrendx

Copy link
Copy Markdown
Member

/te-ci pytorch

Signed-off-by: Sangkug Lym <slym@nvidia.com>
@erhoo82

Copy link
Copy Markdown
CollaboratorAuthor

Can you trigger the CI test again?

Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
@ksivaman

Copy link
Copy Markdown
Member

/te-ci pytorch

@ksivamanksivaman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
@ksivaman

Copy link
Copy Markdown
Member

/te-ci pytorch

@ksivaman
ksivaman merged commit 5d34b2a into NVIDIA:mainApr 12, 2024
pggPL pushed a commit to pggPL/TransformerEngine that referenced this pull request May 23, 2024
* Add LN margin to inference
Signed-off-by: Sangkug Lym <slym@nvidia.com>
* cleanup
Signed-off-by: Sangkug Lym <slym@nvidia.com>
* Fix symbolic func registration
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
* Fix grads
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
---------
Signed-off-by: Sangkug Lym <slym@nvidia.com>
Signed-off-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Co-authored-by: Kirthi Shankar Sivamani <ksivamani@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Sign up for freeto 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.

3 participants

@erhoo82@ptrendx@ksivaman