-
Notifications
You must be signed in to change notification settings - Fork 364
❓ [Question] dynamc engines & interpolation align_corners=True #2327
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
Comments
BTW, there is no setup.py in py folder in main branch. |
Hello - #2146 removed this warning, so it seems the latest |
@gs-olive I uninstall origin torch_tensorrt in docker, and then run
Could you give some help? BTW, I tried
|
This is likely a Torch version mismatch error. Could you instead try, from the root of the directory: pip install . --extra-index-url https://download.pytorch.org/whl/nightly/cu121 --extra-index-url https://pypi.ngc.nvidia.com Alternatively, you can get a Docker container with the latest docker pull ghcr.io/pytorch/tensorrt/torch_tensorrt:main |
@gs-olive Thanks for your help, I tried your docker image, but still get error when importing torch_tensorrt 😂.
|
Thanks for letting me know - it seems there is a mismatch in the |
@gs-olive Looking forward to the solution. 🚀 |
I just checked and the latest deployment of |
@gs-olive Now getting the cuda error when using gpu.😂
|
Hi @ArtemisZGL - I have seen this issue before - when running the docker container, do you specify docker run --rm -it --gpus all ghcr.io/pytorch/tensorrt/torch_tensorrt:main This might help with exposing the GPUs through the docker container. If you prefer to run the latest versions of Torch-TRT outside of a Docker container, you can install the nightly through pip, like so (switch pip install --pre torch torch_tensorrt --extra-index-url https://download.pytorch.org/whl/nightly/cu121 |
@gs-olive Sorry for late reply cause having a long vocation. I used
|
@gs-olive I think that problem is related to driver version, and I reinstall by your command with cu113.
But I met another problem with dynamic shape, here is the code I used to convert the model which should work for dynamic shape accordings to docs.
The model have 3 inputs, while the 1st and 2nd are dynamic but the 3rd is static. After conversion, It can infer the shape with 512 properly, but other shape get shape errors. I also tried to set the 3rd input with min(opt max) shape but still got the same error. Is that I missing somethings?
|
Thanks for the follow-up. I don't believe there should be an issue with mixed static/dynamic inputs. Could you also try specifying |
@gs-olive Sorry for late reply, I got this ir type error when adding
|
What version of |
@gs-olive Opps, I reinstall
BTW, I also tried with |
Thanks for testing that out. When you try |
@gs-olive Sorry for late reply, my torch and torch_tensorrt are from wheels build in 20231012 and still got
|
I think the issue may be in the input model type, then. If you are passing in a TorchScript |
hi! Does your interpolation target size depend on the input size? I have also been trying to run through fully dynamic shape support recently |
❓ Question
What you have already tried
I used the latest docker with tag 23.08-py3. When converting model doing interpolation with align_corners=True and dynamic input, I got error as below.
And I found this check did exist in code with tag v1.4.0, but not in main branch. Will I need to clone the latest code and recompile torch-tensorrt to escape frome this error and will it work? Or any other simple way ?
Environment
nvcr.io/nvidia/pytorch:23.08-py3
Additional context
The text was updated successfully, but these errors were encountered: