-
Notifications
You must be signed in to change notification settings - Fork 365
feat: add convert_method_to_trt_engine()
for dynamo
#2467
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
feat: add convert_method_to_trt_engine()
for dynamo
#2467
Conversation
@gs-olive @narendasan Another question - Regarding testing, how to invoke this function like directly calling |
I think it could potentially go in |
a6ba374
to
9df10a4
Compare
@gs-olive Got a Python package import error:
Can we add |
Hi @zewenli98 - sure, I think we can add this dependency at the path |
a0053c5
to
7e94305
Compare
Thanks! Naren suggested that |
7e94305
to
38caf26
Compare
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.
Overall looks good. Added one suggestion and needs a rebase.
e5a8d62
to
4cbad77
Compare
Description
Add a function
convert_method_to_trt_engine()
for dynamo.I'm not familiar with this part, so detailed reviews and suggestions are greatly welcome.
Fixes #2410
Type of change
Checklist: