Skip to content

Added identity layer#17291

Merged
copybara-service[bot] merged 2 commits intokeras-team:masterfrom
AdityaKane2001:identity3
Dec 5, 2022
Merged

Added identity layer#17291
copybara-service[bot] merged 2 commits intokeras-team:masterfrom
AdityaKane2001:identity3

Conversation

@AdityaKane2001
Copy link
Copy Markdown
Contributor

@fchollet

Please see #17251.

Copy link
Copy Markdown
Collaborator

@fchollet fchollet left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@google-ml-butler google-ml-butler bot added kokoro:force-run ready to pull Ready to be merged into the codebase labels Nov 30, 2022
@AdityaKane2001
Copy link
Copy Markdown
Contributor Author

@fchollet

Resolved the CI build error. Please run the workflow again.

@google-ml-butler google-ml-butler bot added the keras-team-review-pending Pending review by a Keras team member. label Dec 1, 2022
@google-ml-butler google-ml-butler bot removed the ready to pull Ready to be merged into the codebase label Dec 1, 2022
@gbaned gbaned requested review from fchollet and removed request for fchollet December 1, 2022 15:08
@google-ml-butler google-ml-butler bot added kokoro:force-run ready to pull Ready to be merged into the codebase labels Dec 1, 2022
@fchollet fchollet removed the keras-team-review-pending Pending review by a Keras team member. label Dec 1, 2022
@AdityaKane2001
Copy link
Copy Markdown
Contributor Author

@fchollet

Not sure what's causing the build to fail. Here's the error:

ImportError: cannot import name 'layers' from partially initialized module 'keras' (most likely due to a circular import) (/home/kbuilder/.cache/bazel/_bazel_kbuilder/31d6f47147b75c35404d734345be7323/execroot/org_keras/bazel-out/k8-opt/bin/keras/utils/data_utils_test.runfiles/org_keras/keras/__init__.py)

All tests failed with this error.

@copybara-service copybara-service bot merged commit 83fb584 into keras-team:master Dec 5, 2022
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Dec 5, 2022
Imported from GitHub PR keras-team/keras#17291

@fchollet

Please see #17251.
Copybara import of the project:

--
356eaa8a7b035dcac2a565f9dde38201334f1c0c by Aditya Kane <64411306+AdityaKane2001@users.noreply.github.com>:

Added identity layer
--
b8240ece60fced29c0e47970219c064bed18ae83 by Aditya Kane <adityakane1@gmail.com>:

added identity to core import

Merging this change closes #17291

PiperOrigin-RevId: 492867838
@fchollet
Copy link
Copy Markdown
Collaborator

fchollet commented Dec 5, 2022

Just an FYI: I have merged the PR, but it turned out to break a Google test for unknown reasons (seems related to the fact that some users created their own Identity layer in the past) and the change had to be rolled back. I am looking into the issue. Hopefully I can restore the change.

@fchollet
Copy link
Copy Markdown
Collaborator

fchollet commented Dec 7, 2022

I have re-applied the change. One modification I had to do is to apply a tf.identity() op in the call method.

@AdityaKane2001
Copy link
Copy Markdown
Contributor Author

@fchollet Thanks a lot for the follow up. Really nice of you to keep me in the loop. Thoroughly appreciate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to pull Ready to be merged into the codebase size:S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants