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

refactor: _log_epoch_metrics() #18274

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Conversation

arjun-234
Copy link
Contributor

In this refactored version, I have changed the code to consolidate two loops into one loop for the same method, improving efficiency and readability.

In this refactored version, I have changed the code to consolidate two loops into one loop for the same method, improving efficiency and readability.
@google-cla
Copy link

google-cla bot commented Jul 3, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@gbaned gbaned requested a review from rchao July 3, 2023 08:45
@sachinprasadhs sachinprasadhs added the keras-team-review-pending Pending review by a Keras team member. label Jul 5, 2023
@mihirparadkar mihirparadkar removed the keras-team-review-pending Pending review by a Keras team member. label Jul 6, 2023
@rchao
Copy link
Contributor

rchao commented Jul 6, 2023

Thanks for the PR -- now running tests to check this doesn't break anything.

@sachinprasadhs sachinprasadhs added the ready to pull Ready to be merged into the codebase label Jul 14, 2023
copybara-service bot pushed a commit that referenced this pull request Jul 17, 2023
Imported from GitHub PR #18274

In this refactored version, I have changed the code to consolidate two loops into one loop for the same method, improving efficiency and readability.
Copybara import of the project:

--
6a05407 by Arjun D <[email protected]>:

refactor:  _log_epoch_metrics()

In this refactored version, I have changed the code to consolidate two loops into one loop for the same method, improving efficiency and readability.

Merging this change closes #18274

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18274 from arjun-234:master 6a05407
PiperOrigin-RevId: 548715375
@copybara-service copybara-service bot merged commit 151a7bd into keras-team:master Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review 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.

6 participants