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

Group Convolution Support #137

Closed
will-rice opened this issue Jun 7, 2021 · 6 comments
Closed

Group Convolution Support #137

will-rice opened this issue Jun 7, 2021 · 6 comments

Comments

@will-rice
Copy link

Any plan on supporting group convolutions in this repo? I would be interested in working on this also. Just wanted to verify it wasn't already in progress before I attempted it.

@advaitjain
Copy link
Member

In general, the recommendation is to first implement the feature in TfLite and then port to TFLM for embedded use-cases.

I see that you have found and commented on tensorflow/tensorflow#40044 and I would encourage you to follow up there for now.

@will-rice
Copy link
Author

Thank you!!

@MATTYGILO
Copy link

Are they supported yet?

@HemanthSai7
Copy link

HemanthSai7 commented Sep 16, 2024

Hello, are the grouped convolutions supported yet? I'm still facing the issue in using grouped convolutions. I'm able to deploy normal conv layers without any issue. In the TF issue mentioned above, it seems to have been added.

@mansnils
Copy link
Contributor

Hello @HemanthSai7,
At least the CMSIS-NN kernel supports it:

const int groups = input->dims->data[3] / filter->dims->data[3];

@HemanthSai7
Copy link

HemanthSai7 commented Sep 19, 2024

I'm using conv1d in esp32 and I don't think it's there. I'll checkout the link. Thanks

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

5 participants