Skip to content
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

torch_tensorrt compilation fails #80

Open
FredHaa opened this issue Dec 20, 2021 · 0 comments
Open

torch_tensorrt compilation fails #80

FredHaa opened this issue Dec 20, 2021 · 0 comments

Comments

@FredHaa
Copy link

FredHaa commented Dec 20, 2021

Hi

Thank you for contributing this amazing repo!

I've tried to compile the model with the new torch_tensorrt torchscript compiler.
However, it does not seem to support a variable number of arguments which is heavily used for the various forward() functions in the model definition:

torch.jit.frontend.NotSupportedError: Compiled functions can't take variable number of arguments or use keyword-only arguments with defaults:
File "/home/frederik/.local/lib/python3.8/site-packages/performer_pytorch/performer_pytorch.py", line 576
    def forward(self, x, **kwargs):
                          ~~~~~~~ <--- HERE
        if self.auto_check_redraw:
            self.proj_updater.redraw_projections()

If I were to make a pull request which removes the **kwargs in order to provide TensorRT and torchscript support, would it be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant