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

Added groupwiseconv and modified depthwise conv for common interface #948

Closed
wants to merge 2 commits into from

Conversation

arhik
Copy link

@arhik arhik commented Nov 30, 2019

I am trying to add groupcount as a member for drafted changes here ->
FluxML/NNlib.jl#146 (comment)

In short I am following @staticfloat 's suggestion here JuliaGPU/CuArrays.jl#388 (comment)

@@ -51,6 +51,7 @@ function (c::Conv)(x::AbstractArray)
# TODO: breaks gpu broadcast :(
# ndims(x) == ndims(c.weight)-1 && return squeezebatch(c(reshape(x, size(x)..., 1)))
σ, b = c.σ, reshape(c.bias, map(_->1, c.stride)..., :, 1)
@show b
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ignore this. I will remove it.

@DhairyaLGandhi
Copy link
Member

Looking forward to this! Thanks

@arhik arhik marked this pull request as ready for review December 8, 2019 07:38
@ToucheSir
Copy link
Member

Following the breadcrumbs so this issue stays updated, per JuliaGPU/CuArrays.jl#523 (comment) NNlib needs updating before we can revisit a layer in Flux.

@ToucheSir
Copy link
Member

As of #1531, grouped convolutions are now possible with Conv(...; groups=...).

@ToucheSir ToucheSir closed this Jan 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants