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

A significantly large number of queries of the NLQ task have a response window of 0 seconds. #14

Open
md-mohaiminul opened this issue Apr 11, 2022 · 2 comments

Comments

@md-mohaiminul
Copy link

Thanks for this wonderful work!

A significantly large number of queries of the NLQ task have a response window of 0 seconds according to the annotations provided. Particularly, 1304 (8.59%) windows of the train and validation sets have 0s duration. According to the standard evaluation method (https://github.com/EGO4D/episodic-memory/blob/main/NLQ/VSLNet/utils/evaluate_ego4d_nlq.py), these windows will always produce an IoU of 0 irrespective of the predictions. Is this a problem with the dataset?

Moreover, a large number of windows have very small durations. Particularly, 2641 (17.41%) windows of the train and validation sets have a duration of less than 1 second.

I was wondering if these are errors with annotations? How should we handle these cases?

@md-mohaiminul md-mohaiminul changed the title Many response windows for a query of the NLQ task have a 0 second length. A significantly large number of queries of the NLQ task have a response window of 0 seconds. Apr 11, 2022
@ebyrne
Copy link
Collaborator

ebyrne commented May 19, 2022

@md-mohaiminul We can confirm that this is in fact an issue with the annotation. We won't adjust the annotations while the current challenge round is running (till June 1), but we will update the annotations in the near future. We apologize for the additional noise here!

@ebyrne
Copy link
Collaborator

ebyrne commented May 19, 2022

We'll post more updates here as well as they're available: https://discuss.ego4d-data.org/t/nlq-annotation-zero-temporal-windows/36

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