Skip to content

Cherrypick for RandomContrast update#16435

Merged
qlzh727 merged 6 commits intokeras-team:r2.9from
qlzh727:r2.9
Apr 18, 2022
Merged

Cherrypick for RandomContrast update#16435
qlzh727 merged 6 commits intokeras-team:r2.9from
qlzh727:r2.9

Conversation

@qlzh727
Copy link
Copy Markdown
Member

@qlzh727 qlzh727 commented Apr 18, 2022

Piper cl: 439938098

qlzh727 and others added 6 commits April 7, 2022 15:55
PiperOrigin-RevId: 438858608
This is one test file failing, due to the monkey patching happens in the dtensor.init(), and I will need to dig more about the root cause (probably due to patching tf.Variable with DVariable, and cause logic difference for instance type checking.)

PiperOrigin-RevId: 439676157
1. Add docstring for LayoutMap.
2. Hide certain methods for keras.dtensor.optimizers.

PiperOrigin-RevId: 442651549
@qlzh727 qlzh727 requested a review from fchollet April 18, 2022 22:56
@google-ml-butler google-ml-butler bot added the keras-team-review-pending Pending review by a Keras team member. label Apr 18, 2022
@qlzh727 qlzh727 requested a review from LukeWood April 18, 2022 22:56
@LukeWood
Copy link
Copy Markdown
Contributor

Weird CI Is failing. Any idea why?

@qlzh727
Copy link
Copy Markdown
Member Author

qlzh727 commented Apr 18, 2022

The CI failing was expected, since the CI build was configured to test against tf-nightly, and we didn't cherrypick the fix on keras master branch to r2.9. The same test will be run again when we make the release package.

@qlzh727 qlzh727 merged commit 6c1c410 into keras-team:r2.9 Apr 18, 2022
@qlzh727 qlzh727 removed the keras-team-review-pending Pending review by a Keras team member. label Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants