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.

Sign up for GitHub

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

Jump to bottom

Export keras.mixed_precision.DTypePolicy as keras.mixed_precision.Policy #468

Closed
abheesht17 opened this issue Jul 13, 2023 · 2 comments · Fixed by #470
Closed

Export keras.mixed_precision.DTypePolicy as keras.mixed_precision.Policy #468

abheesht17 opened this issue Jul 13, 2023 · 2 comments · Fixed by #470

Comments

@abheesht17
Copy link
Collaborator

abheesht17 commented Jul 13, 2023 •
edited
Loading

In Keras 2.0, we call it Policy: https://keras.io/api/mixed_precision/policy/. Was the rename to DTypePolicy intentional? We can probably export it as Policy and DTypePolicy both (if DTypePolicy is intentional).
@fchollet
Copy link
Contributor

Sure, let's add the alias in the export statement for backwards compatibility. Can you open a PR?

@abheesht17
Copy link
Collaborator Author

Sure thing!

@abheesht17 abheesht17 mentioned this issue Jul 13, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Export DTypePolicy as Policy for backward compat abheesht17/keras-core
2 participants
@fchollet @abheesht17