-
Notifications
You must be signed in to change notification settings - Fork 396
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
fix: 🚨Additional commits for issue #778 #780
Conversation
Thanks for the PR! I will take a careful review later to make sure no bug was introduced. |
a8fbc4f
to
674b640
Compare
This reverts commit 6cf72af.
Codecov ReportBase: 95.02% // Head: 95.02% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #780 +/- ##
==========================================
- Coverage 95.02% 95.02% -0.01%
==========================================
Files 48 48
Lines 3137 3073 -64
==========================================
- Hits 2981 2920 -61
+ Misses 156 153 -3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
Thanks for the update! After addressing these comments, it should be good to merge.
a6440d0
to
e094dc4
Compare
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.
Looks all good! Thank you!
Fixes: keras-team#783 * test: ✏️ Removed `return` Fix also `return` issue as discussed keras-team#780 (comment)
Fixes: keras-team#783 * test: ✏️ Removed `return` Fix also `return` issue as discussed keras-team#780 (comment) * fix: ✏️ Correct link
Objective
During taking care about the
flake8
issue #778,keras_tuner
for cross-checkedvia
for refactoring possibilities:
See also: