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

Correct buffer donation of Optimizer._step. #21

Merged
merged 1 commit into from
May 27, 2022
Merged

Conversation

copybara-service[bot]
Copy link

Correct buffer donation of Optimizer._step.

Buffers can only be donated if they match the shape and type of the output,
which is not true for the rng state or the batch item.

@google-cla
Copy link

google-cla bot commented May 25, 2022

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.

Buffers can only be donated if they match the shape and type of the output,
which is not true for the rng state or the batch item.

PiperOrigin-RevId: 451356262
@copybara-service copybara-service bot merged commit 21201e3 into main May 27, 2022
@copybara-service copybara-service bot deleted the test_451009854 branch May 27, 2022 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants