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

Update indentation level - losses.py #18043

Merged

Conversation

Frightera
Copy link
Contributor

This PR changes the indentation level of the docstrings to 4 spaces.

@gbaned gbaned requested a review from rchao April 24, 2023 04:13
@google-ml-butler google-ml-butler bot added the keras-team-review-pending Pending review by a Keras team member. label Apr 24, 2023
@mihirparadkar mihirparadkar removed the keras-team-review-pending Pending review by a Keras team member. label Apr 26, 2023
@google-ml-butler google-ml-butler bot added the keras-team-review-pending Pending review by a Keras team member. label Apr 27, 2023
@google-ml-butler google-ml-butler bot added kokoro:force-run ready to pull Ready to be merged into the codebase labels Apr 27, 2023
@haifeng-jin haifeng-jin removed the keras-team-review-pending Pending review by a Keras team member. label Apr 27, 2023
copybara-service bot pushed a commit that referenced this pull request Apr 27, 2023
Imported from GitHub PR #18043

This PR changes the indentation level of the docstrings to 4 spaces.
Copybara import of the project:

--
258e866 by Kaan Bıçakcı <[email protected]>:

Update docstring indendation to 4 spaces.

--
7c4ca54 by Kaan Bıçakcı <[email protected]>:

Move CosineSimilarity class to up top.

--
643b1ae by Kaan Bıçakcı <[email protected]>:

Fix formatting

--
8addc40 by Kaan Bıçakcı <[email protected]>:

Add Defaults to `False` for use_legacy_format param.

Merging this change closes #18043

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18043 from Frightera:migrate_losses_docstring 8addc40
PiperOrigin-RevId: 527630600
@google-ml-butler google-ml-butler bot removed the ready to pull Ready to be merged into the codebase label May 2, 2023
@Frightera
Copy link
Contributor Author

Hi @haifeng-jin, some CI checks were failing, updated my branch

@Frightera Frightera requested a review from haifeng-jin May 2, 2023 17:14
@gbaned gbaned added the keras-team-review-pending Pending review by a Keras team member. label May 4, 2023
@google-ml-butler google-ml-butler bot added kokoro:force-run ready to pull Ready to be merged into the codebase labels May 4, 2023
copybara-service bot pushed a commit that referenced this pull request May 4, 2023
Imported from GitHub PR #18043

This PR changes the indentation level of the docstrings to 4 spaces.
Copybara import of the project:

--
258e866 by Kaan Bıçakcı <[email protected]>:

Update docstring indendation to 4 spaces.

--
7c4ca54 by Kaan Bıçakcı <[email protected]>:

Move CosineSimilarity class to up top.

--
643b1ae by Kaan Bıçakcı <[email protected]>:

Fix formatting

--
8addc40 by Kaan Bıçakcı <[email protected]>:

Add Defaults to `False` for use_legacy_format param.

Merging this change closes #18043

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18043 from Frightera:migrate_losses_docstring b305e0a
PiperOrigin-RevId: 527630600
@divyashreepathihalli divyashreepathihalli removed the keras-team-review-pending Pending review by a Keras team member. label May 5, 2023
copybara-service bot pushed a commit that referenced this pull request May 5, 2023
Imported from GitHub PR #18043

This PR changes the indentation level of the docstrings to 4 spaces.
Copybara import of the project:

--
258e866 by Kaan Bıçakcı <[email protected]>:

Update docstring indendation to 4 spaces.

--
7c4ca54 by Kaan Bıçakcı <[email protected]>:

Move CosineSimilarity class to up top.

--
643b1ae by Kaan Bıçakcı <[email protected]>:

Fix formatting

--
8addc40 by Kaan Bıçakcı <[email protected]>:

Add Defaults to `False` for use_legacy_format param.

Merging this change closes #18043

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18043 from Frightera:migrate_losses_docstring b305e0a
PiperOrigin-RevId: 527630600
@copybara-service copybara-service bot merged commit 000b9eb into keras-team:master May 9, 2023
@Frightera Frightera deleted the migrate_losses_docstring branch May 9, 2023 07:03
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:XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants