-
Notifications
You must be signed in to change notification settings - Fork 362
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
fx2trt converters transformer encoder #1814
Merged
apbose
merged 19 commits into
bose_fx2trt_converters_slice_select
from
bose_fx2trt_converters_transformer_encoder
Apr 21, 2023
Merged
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
f1098f2
feat: Add sample torch.compile backend for tensorrt aten path
gs-olive 243bf9b
Add decompositions to aot call
gs-olive 76fd3c8
Mark FX2TRT converter as fake tensor unsupported
gs-olive 6a8102c
Minor naming bugfix
gs-olive 4d7f83d
Merge branch 'main' into bose_fx2trt_converters_transformer_encoder
apbose 205b321
Merge branch 'bose_fx2trt_converters_slice_select' into bose_fx2trt_c…
apbose e97ed50
Implementing aten::chunk, aten::layer_norm, aten::softmax, aten::wher…
apbose c5a4744
Transformer operator changes
apbose 1d78f43
feat: Add ts converter support for aten::all.dim (#1840)
mfeliz-cruise d9545be
Merge branch 'main' into bose_fx2trt_converters_transformer_encoder
apbose 89f2fcf
Merge branch 'bose_fx2trt_converters_slice_select' into bose_fx2trt_c…
apbose ce7f122
Correcting rsqrt and rsub operator
apbose 30c5fd6
python linting issues and removing chunk test
apbose 7ab071d
Correcting acc squeeze test
apbose 36ac0cf
test_reshape expected ops aten.reshape since aten.view has been remov…
apbose eb851b1
removing aten.view in lowering pass
apbose 6b234e0
layer_norm test
apbose 95c1ada
correcting linting error
apbose 1a1b809
correcting dynamic shape layer norm
apbose File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need a discussion on the development plan for this aten2ait. We hope that acc_ops_converters.py are not changed fundamentally unless it is a feature support.
I suggest a good way as Apurba can work on the
aten_ops_converters.py
and call the converters defined in operators.py. If you can reuse any existing implementations inacc_ops_converters.py
, that is good and saves your efforts. If you have a better idea to reuse or need to restructure the code, you can create the utility functions in operator.pySo in way, Apurba's development will lay in
aten_ops_converters.py
and 'operator.py'cc @narendasan @wushirong @yinghai
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, we can sync up and discuss, that would be good!