search

DayBreak-u / darknet_face_with_landmark

加入关键点的darknet训练框架，轻量级的人脸检测，支持ncnn推理
Other
210 stars 71 forks source link

这里 i 是不是有问题？ #18

Open ysc703 opened 4 years ago

ysc703 commented 4 years ago

感谢楼主分享哈^_^ 尝试把代码合到darknet最新版本，发现这里的循环又用到了i https://github.com/ouyanghuiyu/darknet_face_with_landmark/blob/2db381ee665fd16ae0dda62749e5411523371e97/src/yolo_layer.c#L724

DayBreak-u commented 4 years ago

@ysc703 这里应该是不影响的吧， 但是确实需要规范一下

ysc703 commented 4 years ago

哈哈，不好说，因为i值确实是改变了。是不是会改变它上层循环的i，可以调试看看

ysc703 commented 4 years ago

还有770行 no_iou_class_loss_delta[index + (14 + i ) * stride] = 0; no_iou_class_loss_delta在749行被释放了

还是说，这里应该是no_land_class_loss_delta？？？

xiaowenhe commented 4 years ago

@ysc703,770行是错了吗？需要改成no_land_class_loss_delt吗？

又看了下，应该是no_land_class_loss_delt。可能是大佬笔误了。