keras-nlp icon indicating copy to clipboard operation
keras-nlp copied to clipboard

Published 20 hours ago • verified publisher icon keras-team

add YOLOV8

Open divyashreepathihalli opened this issue 1 year ago • 4 comments

This task is part of the KerasHub project ( Consolidating all of KerasCV and KerasNLP models in to one place) Models from KerasCV will be added to KerasNLP in KerasNLP infrstructure style. The development is now under the keras-hub branch in KerasNLP - https://github.com/keras-team/keras-nlp/tree/keras-hub

Steps to complete this Task

  • Source code : https://github.com/keras-team/keras-cv/tree/master/keras_cv/src/models/object_detection/yolo_v8
  • Destination : A model folder here https://github.com/keras-team/keras-nlp/tree/keras-hub/keras_nlp/src/models
  • Example PR : https://github.com/keras-team/keras-nlp/pull/1737, please follow the style from here and take a look at the comments on the PR for any design clarifications
  • copy the model over
  • use KerasNLP test routines to update the tests
  • for reformatting the code run shell/api_gen.sh, shell/format.sh and shell/lint.sh
  • The backbone will be added as a model, any downstream task model will be under the same model folder.

Add either @divyashreepathihalli, @mattdangerw or @SamanehSaadat for review of any PR.

divyashreepathihalli avatar Aug 08 '24 22:08 divyashreepathihalli

Hi @divyashreepathihalli, I would love to work on this. Please assign me. Thank you!

cosmo3769 avatar Aug 16 '24 15:08 cosmo3769

@divyashreepathihalli Yolo_V8 is deeply integrated with bounding_box api of keras_cv. Also calls for a separate loss function CiOU here .

I think this issue could be divided into two separate issues.

Also when I was looking at code, it made sense to me to move bounding boxes and CiOU loss to Keras library rather than in keras_hub. But we can the continue discussion.

I am happy to pick this up.

shanky-kapoor avatar Feb 09 '25 23:02 shanky-kapoor

@divyashreepathihalli , I was thinking to pick this up and start working :)

cheachu avatar Mar 08 '25 05:03 cheachu

@divyashreepathihalli , I was thinking to pick this up and start working :)

I worked on it for a while but there is already a PR https://github.com/keras-team/keras-hub/pull/1899 open for this issue.

Sohaib-Ahmed21 avatar Mar 08 '25 05:03 Sohaib-Ahmed21

Hey all!! Because of license issues, we have decided to drop this.

divyashreepathihalli avatar Jul 10 '25 18:07 divyashreepathihalli

@divyashreepathihalli What does that mean regarding license issues? Does it mean we cannot use YOLOv8 from KerasCV for commercial projects?

hulkds avatar Aug 25 '25 14:08 hulkds

Hey all!! Because of license issues, we have decided to drop this.

@divyashreepathihalli I'm confused. Do you mean the PR #1899 will not be merged? Is there any license issue using KerasCV YOLOv8?

sam31046 avatar Sep 05 '25 01:09 sam31046