search

qqwweee / keras-yolo3

A Keras implementation of YOLOv3 (Tensorflow backend)
MIT License
7.14k stars 3.44k forks source link

Consider if Tiny yolo loss function is wrong #638

Closed GiangHLe closed 4 years ago

GiangHLe commented 4 years ago

https://github.com/qqwweee/keras-yolo3/blob/e6598d13c703029b2686bc2eb8d5c09badf42992/yolo3/model.py#L364

The anchor_mask of tiny_yolo was wrong, right? It should be [[3,4,5],[0,1,2]]

coldrappit commented 4 years ago

you are right. It should be [[3,4,5],[0,1,2]]

robisen1 commented 4 years ago

Did the author react to this? Does he share access with anyone else so they can update code and resolve issues?

GiangHLe commented 4 years ago

no, you can folk it then pull request. However, I haven't seen the author for a long time, I think he dropped this repo and this is a simple bug that we can easy to realize and fix it by ourselves.

robisen1 commented 4 years ago

Yes I understand that it can be forked. A good number have. The problem is they are all different changes made to the code base. A central repo would be great. That being said it is what it is.