Skip to content

Conversation

WindQAQ
Copy link
Member

@WindQAQ WindQAQ commented Sep 17, 2019

  • documentation for lazy adam

@WindQAQ WindQAQ requested a review from a team September 17, 2019 15:32
Copy link
Member

@Squadrick Squadrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for your contribution.

@Squadrick Squadrick merged commit 0dc81fa into tensorflow:master Sep 17, 2019
tomerk pushed a commit to tomerk/addons that referenced this pull request Sep 17, 2019
facaiy pushed a commit that referenced this pull request Sep 19, 2019
* Namespaced all of the custom ops

* Updated C++ namespaces to not conflict w/ TF contrib ones

* Ran code reformatting tool

* Port bug fix in TF contrib to addons. (#497)

* Port bug fix in TF contrib to addons.

Original change at
tensorflow/tensorflow@a913689.

* Fix lint warning.

* check pass through and do the expand_dims() only if needed (#464)

* check pass through and do the expand_dims() only if needed
* add indent to the fixed line
* merge return condition to if state

* add hardshrink kernel (#500)

* add hardshrink kernel
* make linter happy

* Fixing SequenceLoss Keras incompatibility (#503)

* Fix SequenceLoss incompatibility with Keras built-in loops

* Remove debugging prints

* Change the attribute existence checking to use more pythonic way

* Replace some compat.v1 APIs by their v2 equivalent (#507)

* Replace some compat.v1 APIs by their v2 equivalent

* Fix lint error

* Add documentation for LazyAdam (#515)

* Updated hardshrink custom ops & made #ifdef names more consistent.

* Fix to undef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants