-
Notifications
You must be signed in to change notification settings - Fork 611
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
Added layer_test in addons and refactoring the code #1715
Conversation
|
@gabrieldemarmiesse , I have added codes as per you comment. But still many tests fails. Tests fails because of lib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need to use a keras model here. Can you try that and see if the NotFoundError is still there? Thanks a lot
cc @bhack |
Thank you for your contribution. We sincerely apologize for any delay in reviewing, but TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision: Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA: |
We need to modify code in many files to resolve issue #1567 .
This PR adds
layer_test
intesting_utils
of addons.@gabrieldemarmiesse - I have removed all codes in the function corresponding to model checking and
set|get_weights()
. Currently, tests will fail because the method doesn't return anything (previously it was returning something likeactual_output
which was part of Sequential API tests part.). Please review the code and guide me to the right direction.