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

Implementation of BasicTAD #2638

Open
wants to merge 10 commits into
base: dev-1.x
Choose a base branch
from
Open

Conversation

makecent
Copy link
Contributor

Motivation

Implement BasicTAD.

BasicTAD's main contribution is not in inventing complex modules, but in utilizing a modular architecture for temporal action detection tasks. As a result, the implementation of BasicTAD can mostly be achieved through code adapted from mmdetection.

While this project does not aim to replicate the original work's high performance, it does aim to implement its workflow.

Modification

Add a new project named basic_tad

Notes

I am only able to train the model in a basic setting due to my limited computing resources. This includes low spatial-temporal resolution, simple test augmentation, and evaluation on THUMOS14 only.

The current code needs to be cleaned up and the anchor-free version of BasicTAD is still in progress. Before proceeding with future work, may I please confirm your suggestion regarding this project?

@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Patch has no changes to coverable lines.

❗ Current head 4c7bff2 differs from pull request most recent head 874970e. Consider uploading reports for the commit 874970e to get more accurate results

📢 Thoughts on this report? Let us know!.

@Dai-Wenxun
Copy link
Collaborator

Excellent job! Could you please push these commits to dev-1.x branch first? Thank you!

@makecent makecent changed the base branch from main to dev-1.x August 11, 2023 12:54
@makecent
Copy link
Contributor Author

@Dai-Wenxun I changed the branch for merging to dev-1.x . But my commits are based on the main branch so there are some unexpected files changed

@cir7 cir7 requested a review from hukkai August 23, 2023 03:35
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.

4 participants