Skip to content
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

Keras 3 migration #1716

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Keras 3 migration #1716

wants to merge 4 commits into from

Conversation

sachinprasadhs
Copy link
Collaborator

No description provided.

Let's checkout the model summary to have a better insight on the model.


```python
model.summary()
```

<div class="k-default-codeblock">
```
Model: "model"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This summary is the old style summary. The example was converted to Keras 3 already, and now it seems to run on Keras 2.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Surprising that it is using Keras 2, the issue reported here has some problem with the plot model rendering #1713, and the original PR created in keras_core does not seem to have migrated to Keras 3 keras-team/keras-core#884

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to pip install keras --upgrade in your environment before generating the files.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was keras_nlp which was installing keras 2 in the script, I removed that part and generated files again.


```
</div>
5/183 [..............................] - ETA: 43:54 - loss: 1.3896 - accuracy: 0.2000
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's still using Keras 2.

Make sure to install keras 3 after everything else is installed.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's still using Keras 2.

Make sure to install keras 3 after everything else is installed.

Hi @sachinprasadhs Any update on this PR? Please. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants