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
Allow std::array as operator argument and return #34399
Conversation
Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/) [ghstack-poisoned]
Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/) ghstack-source-id: 99695358 Pull Request resolved: #34399
💊 Build failures summary and remediationsAs of commit a678c12 (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker. This comment has been revised 17 times. |
Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/) [ghstack-poisoned]
Pull Request resolved: #34399 Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. ghstack-source-id: 99833901 Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/)
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.
It's kind of copy pastey with the list code, but I'm not sure if there's a good way to eliminate it.
Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/) [ghstack-poisoned]
Pull Request resolved: #34399 Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. ghstack-source-id: 100149868 Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/)
Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/) [ghstack-poisoned]
Pull Request resolved: #34399 Custom ops can now take std::array as arguments and return it. This PR also moves the ops in native_functions.yaml that were blocked by this to now `use_c10_dispatcher: full`. ghstack-source-id: 102643208 Differential Revision: [D20315072](https://our.internmc.facebook.com/intern/diff/D20315072/)
This pull request has been merged in deefafb. |
Stack from ghstack:
Custom ops can now take std::array as arguments and return it.
This PR also moves the ops in native_functions.yaml that were blocked by this to now
use_c10_dispatcher: full
.Differential Revision: D20315072