Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

NHWC not supported for convolution, but the docs say that it is. #7811

Closed
danielwaterworth opened this issue Sep 8, 2017 · 6 comments
Closed

Comments

@danielwaterworth
Copy link

The docs say that NHWC is supported for convolution, but it's not.

This is the error I get:

mxnet.base.MXNetError: [11:54:54] src/operator/./convolution-inl.h:116: Check failed: param_.layout.value() == mshadow::kNCW || param_.layout.value() == mshadow::kNCHW || param_.layout.value() == mshadow::kNCDHW Only support NCW, NCHW and NCDHW layout

Here are the docs. I'm using mxnet 0.11.0.

@danielwaterworth danielwaterworth changed the title NHWC not supported, but the docs say it is. NHWC not supported for convolution, but the docs say that it is. Sep 8, 2017
@piiswrong
Copy link
Contributor

It's only supported on GPU

@danielwaterworth
Copy link
Author

Could you update the docs then

@chinakook
Copy link
Contributor

NHWC is not efficient for image data, use NCHW instead.

@danielwaterworth
Copy link
Author

@chinakook, it would be useful to state that in the docs too.

@eric-haibin-lin
Copy link
Member

This document shows you how to contribute to API docs in MXNet. Anyone is welcome to post a patch for this

@ThomasDelteil
Copy link
Contributor

@yzhliu the docs have been updated, can you close please? thanks

@yzhliu yzhliu closed this as completed Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants