med-air / 3DSAM-adapter

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

Does the relative position embedding work? #5

Closed 22TonyFStark closed 1 year ago

22TonyFStark commented 1 year ago

Hello, I notice that you provide the code for the relative position as optional, I wonder have you done some experiments about it? Does it work well? Thanks for your work, it inspires me a lot :)

peterant330 commented 1 year ago

Hello, I notice that you provide the code for the relative position as optional, I wonder have you done some experiments about it? Does it work well? Thanks for your work, it inspires me a lot :)

The code about the relative position is inherited from the original SAM code. As we are focusing on fine-tuning, for those parameters we just use the same setting as the original code without further ablation (which is set to be true if I remember correctly).