oneDNN icon indicating copy to clipboard operation
oneDNN copied to clipboard

Typos in the Memory Format Diagram of Blocked Layout

Open topsy404 opened this issue 1 year ago • 1 comments

Summary

There are two typos in the memory format diagram.

URLs

Blocked Layout Docs Diagram Link

Additional details

  1. Since the sub-block size is 8, the last plane of the first sub-block should be the plane with index 7 instead of the plane with index 8.
  2. Furthermore, the physical data layout in the diagram is incorrect. The [x:3] should switch from [139 159] to [160 180] instead of [159 179] to [180 200].

I think the correct version should be the following:

correct_diagram

topsy404 avatar May 06 '23 11:05 topsy404

Thanks for the catch. Indeed you are correct. We will fix this soon.

mgouicem avatar Jun 06 '23 14:06 mgouicem