-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update base image to 2410 #8164
base: dev
Are you sure you want to change the base?
Conversation
Signed-off-by: YunLiu <[email protected]>
Hi @borisfom, could you please take a look at this KeyError that occurred when I upgraded the base image to version 24.09?
|
@KumoLiu : Unfortunately, this looks like internal Torch-TensorRT error I don't see an easy workaround for. Please just disable the failing test (test_handler) for now. |
Yes, the error originates from the internal code in Torch. However, I'm wondering if we need to update the base image. As you mentioned here, if not, we can simply wait for the 24.10 release. Thanks. |
report a bug here: pytorch/pytorch#138674 |
@KumoLiu : yes we should just wait until 24.10 release |
Signed-off-by: YunLiu <[email protected]>
Signed-off-by: YunLiu <[email protected]>
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.
LGTM. The title should say '2410' though.
/build |
Signed-off-by: YunLiu <[email protected]>
/build |
Fixes # .
Description
A few sentences describing the changes proposed in this pull request.
Types of changes
./runtests.sh -f -u --net --coverage
../runtests.sh --quick --unittests --disttests
.make html
command in thedocs/
folder.