search

med-air / 3DSAM-adapter

Holistic Adaptation of SAM from 2D to 3D for Promptable Medical Image Segmentation
134 stars 12 forks source link

Not so sure about the patch embedding kernel size. #16

Open Asagami-Fujino opened 11 months ago

Asagami-Fujino commented 11 months ago

Thanks a lot for your work，it helps a lot. In your paper you mentioned that the convolution kernel size is a conbination of 1x14x14 and 14x1x1, but in the code it seems that the kernel size is 1x16x16 and 16x1x1. So which is correct?

peterant330 commented 11 months ago

Thanks a lot for your work，it helps a lot. In your paper you mentioned that the convolution kernel size is a conbination of 1x14x14 and 14x1x1, but in the code it seems that the kernel size is 1x16x16 and 16x1x1. So which is correct?

Hi,

It should be 16. There is a typo in the paper. Thanks for pointing this out.