addons icon indicating copy to clipboard operation
addons copied to clipboard

Fixed TypeError in CyclicalLearningRate

Open verenaAI opened this issue 2 years ago • 2 comments

Description

Fixed bug which appears at the end of the first training epoch. With TF2.5 or higher, the optimizer returns the step variable as int64 but the scale_fn function needs it to be of type float for Triangular2CyclicalLearningRate and ExponentialCyclicalLearningRate.

Type of change

Checklist:

  • [ ] I've properly formatted my code according to the guidelines
    • [ ] By running Black + Flake8
    • [ ] By running pre-commit hooks
  • [ ] This PR addresses an already submitted issue for TensorFlow Addons
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?

verenaAI avatar Sep 13 '22 12:09 verenaAI

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 '22 12:09 google-cla[bot]

@raphaelmeudec

You are owner of some files modified in this pull request. Would you kindly review the changes whenever you have the time to? Thank you very much.