Replace insecure keras dependency with tf.keras #12
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
Codex Review: Here are some suggestions.
Reply with @codex fix comments to fix any unresolved comments.
About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".
| ############################################ | ||
|
|
||
| optimizer = Adam(lr=learning_rate, beta_1=0.9, beta_2=0.999, epsilon=1e-08, decay=0.0) | ||
| optimizer = Adam(learning_rate=learning_rate, beta_1=0.9, beta_2=0.999, epsilon=1e-08, decay=0.0) |
There was a problem hiding this comment.
[P0] Remove unsupported decay argument from tf.keras Adam
The optimizer initialization still passes decay=0.0, which was valid for keras.optimizers.Adam but is no longer accepted by tf.keras.optimizers.Adam in the TensorFlow ≥2.15 dependency introduced in this commit. Instantiating the optimizer will raise TypeError: __init__() got an unexpected keyword argument 'decay', preventing the model from compiling or training. Consider dropping the argument or switching to tf.keras.optimizers.legacy.Adam if the legacy API is required.
Useful? React with 👍 / 👎.
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68ce6ebbf21083219a0854616942084d