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

Any improvement? #4

Open
John1231983 opened this issue Aug 10, 2022 · 4 comments
Open

Any improvement? #4

John1231983 opened this issue Aug 10, 2022 · 4 comments

Comments

@John1231983
Copy link

Thanks for sharing the repo. Do you achieve any improvement in comparison with training student only?

@John1231983
Copy link
Author

If I have detect hand, so the number of class will be 1 or 2? Am I right to change it to support hand detection?

            c_obj_scale = t_obj_scale.unsqueeze(-1).repeat(1,1,1,1,2)
            # if dist_loss == "kl":
            kl_loss = DclsLoss(F.log_softmax(pi[..., 5:]/T,dim=-1), F.softmax(t_pi[..., 5:]/T, dim=-1))*(T*T)

@xiao-hua-sheng
Copy link
Owner

Thanks for sharing the repo. Do you achieve any improvement in comparison with training student only?

mAP will increase by 1-2%

If I have detect hand, so the number of class will be 1 or 2? Am I right to change it to support hand detection?

            c_obj_scale = t_obj_scale.unsqueeze(-1).repeat(1,1,1,1,2)
            # if dist_loss == "kl":
            kl_loss = DclsLoss(F.log_softmax(pi[..., 5:]/T,dim=-1), F.softmax(t_pi[..., 5:]/T, dim=-1))*(T*T)

right

@John1231983
Copy link
Author

In your code, you setup it as 5 as number of classes. Could you list 5 classes? Do you count background?

@972821054
Copy link

@John1231983 @xiao-hua-sheng Did you solve this problem? I have the same problem, do I need to count the background class?

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

3 participants