PocketFlow icon indicating copy to clipboard operation
PocketFlow copied to clipboard

A Little Problem in the part of Algorithmic Description (convolutional operation)

Open my777777 opened this issue 6 years ago • 1 comments
trafficstars

  • Doc Link: https://pocketflow.github.io/cpr_learner/

Describe the documentation issue There is a little problem in the algorithm description section. When it comes to "The matrix multiplication can be decomposed along the dimension of input channels. ", the size of sub-matrices {Wi} should probably be hkwk × co, not hkwkci× co, isn't it?

my777777 avatar Mar 22 '19 02:03 my777777

Thanks for pointing this out. You're right. We will change from

We divide $\mathbf{X}$ into $c_{i}$ sub-matrices $\left{ \mathbf{X}{i} \right}$, each of size $N h{o} w_{o} \times h_{k} w_{k}$, and similarly divide $\mathbf{W}$ into $c_{i}$ sub-matrices $\left{ \mathbf{W}{i} \right}$, each of size $h{k} w_{k} c_{i} \times c_{o}$.

to

We divide $\mathbf{X}$ into $c_{i}$ sub-matrices $\left{ \mathbf{X}{i} \right}$, each of size $N h{o} w_{o} \times h_{k} w_{k}$, and similarly divide $\mathbf{W}$ into $c_{i}$ sub-matrices $\left{ \mathbf{W}{i} \right}$, each of size $h{k} w_{k} \times c_{o}$.

jiaxiang-wu avatar Mar 23 '19 02:03 jiaxiang-wu