Errors included in the process of assigning the Position embedding vector for multi-head self attention

[junhyung-SPALab]

Thanks for your work. In the process of studying your great open source, I leave a question.

There seems to be a bug in the process of using the position embedding vector in the operation of the DSVT Block.

It seems that the position embedding vector required for Attention is assigned incorrectly.

The questions have been summarized in the image below.
//////////////////////////////////////////////////////////////////
This is an example of a case where DSVT is composed of one stage and each stage is designed with two blocks.

//////////////////////////////////////////////////////////////////

Thanks.

[chenshi3]

Thank you for raising this issue! We have also identified this bug and documented it in OpenPCDet. The reported results were obtained from this version, and we have chosen not to rectify the bug in the code. Furthermore, I attempted to address this bug but noticed negligible performance enhancements. I will make a note of this bug in the Readme for clarity.

[junhyung-SPALab]

Thanks for confirming my issue. I think your README update will be helpful to other researchers in the future.