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.

Sign up for GitHub

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

Jump to bottom

deformable_conv2d #20945

Closed
Tracked by #14951
themaigod opened this issue Jul 28, 2023 · 1 comment
Closed
Tracked by #14951

deformable_conv2d #20945

themaigod opened this issue Jul 28, 2023 · 1 comment
Labels
Sub Task a sub task which is stemming from a ToDo list issue

Comments

@themaigod
Copy link
Contributor

No description provided.

@themaigod themaigod added the Sub Task a sub task which is stemming from a ToDo list issue label Jul 28, 2023
@themaigod themaigod mentioned this issue Jul 28, 2023
Open
33 tasks
@themaigod
Copy link
Contributor Author

As I have discussed in #20138 , this function is not supported in ivy api and cannot be simply achieved by using conv2d. Achieve example: https://github.com/oeway/pytorch-deform-conv
If backend function supports that, it would usually use a specific low-level operation (such as a cuda operator), see: https://pytorch.org/vision/stable/_modules/torchvision/ops/deform_conv.html#deform_conv2d
https://www.mindspore.cn/docs/en/master/_modules/mindspore/ops/function/nn_func.html#deformable_conv2d

So, I think we should not include this function unless we have an ivy api for this operation.

@themaigod themaigod changed the title <deformable_conv2d> in mindspore frontend <deformable_conv2d> Jul 28, 2023
@themaigod themaigod changed the title <deformable_conv2d> deformable_conv2d Jul 28, 2023
This was referenced Jul 28, 2023
Merged
Closed
@iamjameskeane iamjameskeane linked a pull request Jul 31, 2023 that will close this issue
Add conv ops for mindspore frontend #20138
Merged
@iamjameskeane iamjameskeane removed a link to a pull request Jul 31, 2023
Add conv ops for mindspore frontend #20138
Merged
@themaigod themaigod mentioned this issue Aug 2, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Sub Task a sub task which is stemming from a ToDo list issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@themaigod @ivy-leaves