Christoph Heindl

@abhineet123 thanks for the mwe. Unfortunately I don't have access to matlab right now, so as far as I understand your readme I cannot run the MWE. Could you provide...

Hey, Thanks for the PR. Please ensure that the unit tests pass (they also need to cover the new metrics) to be considered for inclusion. Thanks Ramana Subramanyam schrieb am...

Sure, first try to run the test locally instead of in CI: ``` pytest ``` in the main motmetrics directory should run all tests. This requires `pip install pytest` and...

Thanks! Did you add the metrics to the default motmetrics? Is that meaningful? @jvlmdr what's your opinion on this PR?

Hey thanks for the PR! Couple of comments: - Calling `sort_index` in `new_event_dataframe` is not doing anything as the dataframe is empty. - Did you see any performance improvements? -...

Hey, sorry for the late reply. Yes that seems like an oversight when we added euclidean distance metrics. Would you mind opening a PR?

@FangLintao , @wenbodu971228 I agree, adding HOTA would be a nice addon. I'd be happy to include a PR if someone finds the time. Best, Christoph

Your file structure should match ``` --------- Layout for ground truth data //gt/gt.txt //gt/gt.txt ... Layout for test data /.txt /.txt ... Sequences of ground truth and test will be...

this might be caused by duplicate ids in single frames as reported in #60.

Interesting. I hope to find some time to inspect soon.