Bo Wang
Bo Wang
@Mark-M2L could you please try if this works? https://github.com/pytorch/TensorRT/issues/983
Hey @Mark-M2L, sorry I didn't make it clear. I think what you can do is simply add `torch::jit::exception_elimination` right after this line: https://github.com/pytorch/TensorRT/blob/5d1acbacb3928c7d5b1f125cf8fe98c9bbaffbeb/core/lowering/lowering.cpp#L35. Then recompile Torch-TensorRT, it should be good.
Hey @Mark-M2L, do you have a small repro so I can also run and test locally? Btw, we see this error previously when we have this kind of operations: ```...
Hey @Mark-M2L , I'm going to have a test this week. Stuck on something else last week. Will update to you soon.
Hey @Mark-M2L I run your model locally and I have this bug https://github.com/pytorch/TensorRT/issues/1336. Seems like it's because I'm using latest Torch-TensorRT version while you are using 1.10, and there are...
Hey @Mark-M2L, sorry I forgot to reply earlier. Could you please try this 2 PRs #1263 , #1345 ? I tested locally and your model is supported using these 2...
@Mark-M2L All the 2 PRs mentioned above are merged into master branch, so your model should be supported by master branch now. I'm closing this issue.
> @mfeliz-cruise we are currently working on a major restructuring of the partitioning phase to hopefully bring it closer to other design patterns in the project and make it easier...
This PR should be good to merge. @narendasan @peri044
> Does this solution not change the input names anyway? yes, that's what we expect. Any thought on this? curious about whether it's really necessary or not.