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

Refactor SkipLayerNorm and handle beta properly #22862

Merged
merged 8 commits into from
Nov 17, 2024
Merged

Conversation

liqunfu
Copy link
Contributor

@liqunfu liqunfu commented Nov 16, 2024

No description provided.

Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You can commit the suggested changes from lintrunner.

onnxruntime/contrib_ops/cpu/skip_layer_norm.cc Outdated Show resolved Hide resolved
onnxruntime/contrib_ops/cpu/skip_layer_norm.cc Outdated Show resolved Hide resolved
onnxruntime/contrib_ops/cpu/skip_layer_norm.cc Outdated Show resolved Hide resolved
liqunfu and others added 4 commits November 15, 2024 16:47
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
amarin16
amarin16 previously approved these changes Nov 16, 2024
@sophies927 sophies927 added triage:approved Approved for cherrypicks for release release:1.20.1 labels Nov 16, 2024
@tianleiwu
Copy link
Contributor

Do we have unit test cover this?

@liqunfu liqunfu changed the title refactor and handle beta properly Refactor SkipLayerNorm and handle beta properly Nov 16, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
@liqunfu
Copy link
Contributor Author

liqunfu commented Nov 16, 2024

Do we have unit test cover this?

Not with this PR, shall add test cases in a different PR. Need to merge EOD so not to block the release.

Signed-off-by: Liqun Fu <liqfu@microsoft.com>
amarin16
amarin16 previously approved these changes Nov 17, 2024
@jywu-msft jywu-msft closed this Nov 17, 2024
@jywu-msft jywu-msft reopened this Nov 17, 2024
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
@liqunfu liqunfu merged commit 101ed10 into main Nov 17, 2024
93 checks passed
@liqunfu liqunfu deleted the liqun/fix-skip-layer-norm branch November 17, 2024 22:51
yf711 pushed a commit that referenced this pull request Nov 18, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ishwar-raut1 pushed a commit to ishwar-raut1/onnxruntime that referenced this pull request Nov 19, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
yf711 added a commit that referenced this pull request Nov 19, 2024
### Description
<!-- Describe your changes. -->
All three PRs are cherry-picked in this round:
1. [Refactor SkipLayerNorm and handle beta properly (#22862)
](#22862)
2. [[TensorRT EP] Exclude DDS ops from running on TRT
(#22875)](#22875)
3. [[QNN EP] QNN SDK 2.28.2 (#22844) 
](#22844)
### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

---------

Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: Chi Lo <54722500+chilo-ms@users.noreply.github.com>
Co-authored-by: liqun Fu <liqfu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Adrian Lizarraga <adlizarraga@microsoft.com>
guschmue pushed a commit that referenced this pull request Dec 2, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
Signed-off-by: Liqun Fu <liqun.fu@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:1.20.1 triage:approved Approved for cherrypicks for release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants