Skip to content

Add ActivityNet format - #87

Merged
horatiualmasan merged 9 commits into
mainfrom
horatiu-lig-9805-import-activitynet-style-event-annotations-2
Jul 13, 2026
Merged

Add ActivityNet format#87
horatiualmasan merged 9 commits into
mainfrom
horatiu-lig-9805-import-activitynet-style-event-annotations-2

Conversation

@horatiualmasan

@horatiualmasan horatiualmasan commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What has changed and why?

Add parser for ActivityNet annotations
ground truth annotations example: https://github.com/activitynet/ActivityNet/blob/master/Evaluation/data/activity_net.v1-3.min.json
predictions example: http://activity-net.org/challenges/2016/guidelines.html

How has it been tested?

New tests

@horatiualmasan

Copy link
Copy Markdown
Contributor Author

/review

@JonasWurst JonasWurst left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not clear to me how this will work.

def _parse_activitynet_data(
data: JsonDict,
) -> tuple[list[VideoTemporalClassification], list[Category]]:
if "database" in data:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can it contain both?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be separate.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked a few files: e.g. https://raw.githubusercontent.com/imatge-upc/activitynet-2016-cvprw/refs/heads/master/dataset/activity_net.v1-3.min.json

What I miss in this implementation: we have also video level meta data that is not considered (duration, split). also, how will one be able to connect the video annotations is the video_id enough/how are video_id mapped to paths?

@horatiualmasan
horatiualmasan merged commit f89fe68 into main Jul 13, 2026
3 of 4 checks passed
@horatiualmasan
horatiualmasan deleted the horatiu-lig-9805-import-activitynet-style-event-annotations-2 branch July 13, 2026 14:25
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

Successfully merging this pull request may close these issues.

2 participants