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

Discrepancy in DAS Metric Calculation #3

Open
mrclovvn opened this issue Sep 11, 2024 · 0 comments
Open

Discrepancy in DAS Metric Calculation #3

mrclovvn opened this issue Sep 11, 2024 · 0 comments

Comments

@mrclovvn
Copy link

Hi,

Thank you for sharing your valuable work on the "SC3K" paper presented at ICCV 2023. I greatly appreciate the contributions your team has made to this field. However, I wanted to bring a potential discrepancy to your attention regarding the calculation of the DAS metric in your implementation, which raises concerns about the DAS performance reported in the paper.

The paper mentions that the DAS metric follows the approach described in the CVPR 2021 paper, "Skeleton Merger (SM)." However, after reviewing the code implementation in your repository, I noticed differences compared to the method described in the SM paper. Specifically, in SM, the DAS metric is calculated by evaluating both the forward alignment score and the backward alignment score, based on the consistency of keypoint alignments with manually annotated semantic_ids, and the final DAS is computed by averaging these two scores. This dual alignment process seems to be absent in the current implementation, which also does not use the annotations from the KeypointNet dataset for evaluating DAS.

As a result, the DAS metric output from your code appears to differ from the results in the SM implementation. Clarifying and addressing these differences would enhance the accuracy of the paper and its corresponding implementation.

Please let me know if further clarification is needed or if any additional details would be helpful.

Best regards,
Shawn.

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

1 participant