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

question for create centroid #21

Open
RainFrost1 opened this issue Apr 22, 2022 · 8 comments
Open

question for create centroid #21

RainFrost1 opened this issue Apr 22, 2022 · 8 comments

Comments

@RainFrost1
Copy link

RainFrost1 commented Apr 22, 2022

In the function of validation_create_centroids, the line, inds is just for gallery_camids, but camids is the combination of query and gallery. Maybe selected_camids_g = camids[num_query:][inds]?

@RainFrost1
Copy link
Author

@mikwieczorek

@RainFrost1
Copy link
Author

图片
if use this modification, the result in Maket1501 is as above

@zhaoxin111
Copy link

This bug has too much impact

@zhaoxin111
Copy link

图片 if use this modification, the result in Maket1501 is as above

Is this reuslt evaluated with centroids ?

@mikwieczorek
Copy link
Owner

Hi @RainFrost1, Thank you for digging into the code and noticing the possible problems. I will have a look into it at the earliest possible time and update you on the issue. I need to debug the code and confirm your findings.

@RainFrost1
Copy link
Author

Hi @RainFrost1, Thank you for digging into the code and noticing the possible problems. I will have a look into it at the earliest possible time and update you on the issue. I need to debug the code and confirm your findings.

Thank you for your reply and looking forward your result

@RainFrost1
Copy link
Author

图片 if use this modification, the result in Maket1501 is as above

Is this reuslt evaluated with centroids ?
yes,using the original code with the modification as mentioned above

@danganhtuan3738
Copy link

图片 if use this modification, the result in Maket1501 is as above

Is this reuslt evaluated with centroids ?
yes,using the original code with the modification as mentioned above

how you can run mAP?

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

4 participants