-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrating Evaluating and exporting scikit-learn metrics in a Keras callback example to Keras 3 #1737
base: master
Are you sure you want to change the base?
Migrating Evaluating and exporting scikit-learn metrics in a Keras callback example to Keras 3 #1737
Conversation
The Tensorflow backend worked well. For other backends, validation_split was a problem. So, I tried with validation_data instead but still got an attribute error. If the changes are all approved in the.py file,. Then I will change the rest of the files accordingly, with updated dates. |
What was the error in either case? Can you share a Colab to reproduce? |
For jax: https://colab.research.google.com/drive/1MziZSUWEuseaQ6DL_6ykaY9jDk4I2WpB?usp=sharing Both the Colabs show the problem with |
Thanks for the report. It looks like we only support np arrays and tf tensors for validation_split. This can be fixed easily, actually. Let me just do that in Keras. |
Done. You can retry after building Keras from sources. |
I tried, and the problem of validation split is solved, but errors are still there. For pytorch, You can check the notebooks for errors. |
Just replace |
Thanks, it helped the Jax backend. But pytorch is still giving the error |
I looked over the code again and think the JacardScoreCallback is the cause. Although I attempted, I kept receiving the same error. The only thing causing issues is the torch backend. So, what ought to I do with this PR? Should I point out that the PR currently supports both Jax and TensorFlow backends? cc: @fchollet |
What's the complete stack trace? |
This PR changes the Evaluating and exporting scikit-learn metrics in a Keras callback example to keras 3.0 [TF Only Backend].
For example, here is the notebook link provided:
https://colab.research.google.com/drive/1rG2dhTVwJI4_FV3zgNtaxeq2lF7TN-Y1?usp=sharing
cc: @divyashreepathihalli @fchollet
The following describes the Git difference for the changed files:
Changes: