Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SA-M strategy #5

Open
seacloud-0420 opened this issue Jun 14, 2022 · 5 comments
Open

SA-M strategy #5

seacloud-0420 opened this issue Jun 14, 2022 · 5 comments

Comments

@seacloud-0420
Copy link

Hi, I found that the formula for calculating distance in the code is different from that in the paper. Is the effect of using the formula in the code similar to that in the paper? Also,why is the formula for calculating distance different in [0,90] and other angles?

@houliping
Copy link
Owner

houliping commented Jun 15, 2022 via email

@seacloud-0420
Copy link
Author

Thank you for your answer. I still don't understand.
1、The calculation in the code (distances = (bboxes_points[:, None, :] - gt_points[None, :, :]).pow(2).sum(-1).sqrt() ) is different from that in the paper.
1655301267968
2、Why is the angle converted to [0, pi/2] during calculation. I still don't understand.

@houliping
Copy link
Owner

The code for the calculation of Equation 4 in the paper is shown in the figure, which is consistent with the paper. You can see it in the files SASM/sasm_reppoints/sam_reppoints_head.py and SASM/sasm_s2anet/sam_anchor_target.py.
屏幕快照 2022-06-16 10 08 41

The calculation of angles with pi is more accurate and convenient, which is a common way to handle data.

@seacloud-0420
Copy link
Author

Hi, why the formula for calculating distance weights in the code is different from that in the paper. The paper is
1668954134451, the code is
1668954171184. When the formula in the code is changed to be the same as that in the paper, the effect is very poor and almost does not work.

@seacloud-0420
Copy link
Author

seacloud-0420 commented Nov 21, 2022

@houliping
Hi,please help me answer the above question if it is convenient. Thank you very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants