-
Notifications
You must be signed in to change notification settings - Fork 359
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
Create a system to define converters that are written in python #56
Comments
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
This issue has not seen activity for 30 days, Remove stale label or comment or this will be closed in 5 days |
Summary: Pull Request resolved: https://github.com/pytorch/fx2trt/pull/56 ATT Reviewed By: khabinov Differential Revision: D35649693 fbshipit-source-id: b3ef48142bdbeeca24cf12791f5946d8362d0564
# Description Adds converter support for aten::squeeze(Tensor) which will remove any static dimension of size 1. An existing converter supports aten::squeeze(Tensor, int dim). Fixes # (issue) ## Type of change Please delete options that are not relevant and/or add your own. - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - This change requires a documentation update # Checklist: - [ ] My code follows the style guidelines of this project (You can use the linters) - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas and hacks - [ ] I have made corresponding changes to the documentation - [ ] I have added tests to verify my fix or my feature - [ ] New and existing unit tests pass locally with my changes - [ ] I have added the relevant labels to my PR in so that relevant reviewers are notified
# Description Adds converter support for aten::squeeze(Tensor) which will remove any static dimension of size 1. An existing converter supports aten::squeeze(Tensor, int dim). Fixes # (issue) ## Type of change Please delete options that are not relevant and/or add your own. - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - This change requires a documentation update # Checklist: - [ ] My code follows the style guidelines of this project (You can use the linters) - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas and hacks - [ ] I have made corresponding changes to the documentation - [ ] I have added tests to verify my fix or my feature - [ ] New and existing unit tests pass locally with my changes - [ ] I have added the relevant labels to my PR in so that relevant reviewers are notified
It would be nice if we are able to define converters in python for users to add support for a custom operator
The text was updated successfully, but these errors were encountered: