search

facebookresearch / SlowFast

PySlowFast: video understanding codebase from FAIR for reproducing state-of-the-art video models.
Apache License 2.0
6.65k stars 1.22k forks source link

Error: AVAMeter has no attribute video_preds #259

Closed FabiTheGabi closed 4 years ago

FabiTheGabi commented 4 years ago

Dear SlowFast-Team,

this line of code results into an error, because the AVAMeter-class has no attribute video_preds.

https://github.com/facebookresearch/SlowFast/blob/cec2d0a3f17e5d1528098999300332e6aa1066ce/tools/test_net.py#L107

Thus it would only work, if "not cfg.DETECTION.ENABLE".

It would be awesome, if you would implement this feature also for detection tasks.

May I ask, whether you are planning to add TENSORBORAD.HITOGRAM and TENSORBOARD.CONFUSION_MATRIX for detection tasks?

Thank you a lot for your great work.

FabiTheGabi commented 4 years ago

This line of code results into the same error.

https://github.com/facebookresearch/SlowFast/blob/cec2d0a3f17e5d1528098999300332e6aa1066ce/tools/train_net.py#L256

lequytra commented 4 years ago

Hi @FabiTheGabi,

Thank you for bringing the issue to our attention. We just added a small fix to avoid this error. For the being, we don't have plans to add confusion matrix visualization for detection tasks since it doesn't work quite well with multi-label dataset.

FabiTheGabi commented 4 years ago

Thank you a lot for your quick fix and the explanation 👍