models icon indicating copy to clipboard operation
models copied to clipboard

Published 20 hours ago • verified publisher icon tensorflow

YOLOv4-tiny: Adding config and updating YOLO's README

Open jacob-zietek opened this issue 3 years ago • 3 comments

Description

This PR adds a new YOLOv4-tiny config that achieves 21.21AP. It also updates the YOLO project's README.md to include metrics and a place for checkpoints.

Type of change

  • [x] A new research paper code implementation

Checklist

  • [x] I have signed the Contributor License Agreement.
  • [x] I have read guidelines for pull request.
  • [x] My code follows the coding guidelines.
  • [x] I have performed a self code review of my own code.
  • [x] I have commented my code, particularly in hard-to-understand areas.
  • [x] I have made corresponding changes to the documentation.
  • [x] My changes generate no new warnings.
  • [x] I have added tests that prove my fix is effective or that my feature works.

jacob-zietek avatar Apr 25 '22 22:04 jacob-zietek

@fyangf To confirm, you'd like me to move the changes from

official/vision/beta/projects/yolo/README.md => official/projects/yolo/README.md

and

official/vision/beta/projects/yolo/configs/experiments/yolov4/detection/yolov4_tiny_416_tpu.yaml => official/projects/yolo/configs/experiments/yolov4/detection/yolov4_tiny_416_tpu.yaml?

jacob-zietek avatar Feb 10 '23 21:02 jacob-zietek

@fyangf To confirm, you'd like me to move the changes from

official/vision/beta/projects/yolo/README.md => official/projects/yolo/README.md

and

official/vision/beta/projects/yolo/configs/experiments/yolov4/detection/yolov4_tiny_416_tpu.yaml => official/projects/yolo/configs/experiments/yolov4/detection/yolov4_tiny_416_tpu.yaml?

Yes exactly.

fyangf avatar Feb 10 '23 21:02 fyangf

@fyangf This should work now, please let me know if you need anything else. Thank you!

jacob-zietek avatar Feb 10 '23 21:02 jacob-zietek

@fyangf Is there anything you need on my end?

jacob-zietek avatar May 16 '23 22:05 jacob-zietek

@fyangf Is there anything you need on my end?

Can you sync to master branch again and resolve the conflict? There are some new changes to the README file.

fyangf avatar May 18 '23 16:05 fyangf

@fyangf All done, thanks!

jacob-zietek avatar May 18 '23 17:05 jacob-zietek

Hi @fyangf, is there anything else I need to do to get this merged?

jacob-zietek avatar Jun 23 '23 01:06 jacob-zietek

Hi @fyangf. Anything else I'd need to do to get this merged?

jacob-zietek avatar Sep 12 '23 05:09 jacob-zietek

Hi @jacob-zietek,

Could you please sync to head and push it again.It seems something went wrong internally.

I apologize for the duplication of effort.

Thanks.

laxmareddyp avatar Sep 12 '23 22:09 laxmareddyp

I just synced it, thanks. @laxmareddyp

jacob-zietek avatar Sep 24 '23 02:09 jacob-zietek