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

Add activation function option for yolov8 detector.

Open digital-nomad-cheng opened this issue 1 year ago • 4 comments

What does this PR do?

Currently, only YOLOV8Backbon accept activation as argument for changing to other activation function other than swish. But for some hardware, swish activation function is not well optimized. It does not make sense to only change the backbone to other activation function while keep the detector(head) default to swish. So this pull meant to give option to choose activation in detector as well.

digital-nomad-cheng avatar Sep 13 '24 11:09 digital-nomad-cheng

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

google-cla[bot] avatar Sep 13 '24 11:09 google-cla[bot]

@divyashreepathihalli Can you help have a look at this PR? Thanks!

digital-nomad-cheng avatar Sep 13 '24 11:09 digital-nomad-cheng

Thanks for the PR. There was recent changes to YOLV8. Can you please rebase and also runn the shell/format.sh at root to clear formatting errors

divyashreepathihalli avatar Sep 17 '24 23:09 divyashreepathihalli

Thanks for the PR. There was recent changes to YOLV8. Can you please rebase and also runn the shell/format.sh at root to clear formatting errors

Done

digital-nomad-cheng avatar Sep 18 '24 15:09 digital-nomad-cheng