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

Branch3D #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Branch3D #2

wants to merge 3 commits into from

Conversation

stratosGithub
Copy link

Hi,

I've slightly adapted your code, so it can also be used for calculating the 3D MOT scores. Wanted to use it to benchmark the APIDIS dataset and your code was the only I could find that was easy to read and change. I've basically added a boolean member variable to the class MOTEvaluation, and by setting this variable to true or false it is possible to switch between using the overlap of the bounding boxes and the euclidean distance for calculating the MOTP and MOTA scores. I've also created a new branch for my version of the code, so you can leave your master branch unchanged. So if you think my contribution would be useful would be great if you would add it to your repository.

Stratos

@herr-biber herr-biber mentioned this pull request Mar 13, 2017
@herr-biber
Copy link
Contributor

Thanks for the pull request - and sorry for the long delay. Can you please fix the variable names to fit the existing ones (underscores instead of camel case)? Can you further provide a test case? Then I'll be more than happy to merge the pull request.

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