Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix HybridLambda repr #8565

Merged
merged 2 commits into from
Nov 15, 2017
Merged

Fix HybridLambda repr #8565

merged 2 commits into from
Nov 15, 2017

Conversation

JulianSlzr
Copy link
Contributor

Description

Could not "print" a model containing a HybridLambda block; this fixes the offending typo.

Checklist

Essentials

  • Passed code style checking (make lint)
  • Changes are complete (i.e. I finished coding on this PR)
  • [n/a] All changes have test coverage
  • [n/a] For user-facing API changes, API doc string has been updated.
  • To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@piiswrong piiswrong requested a review from szha November 6, 2017 23:25
Copy link
Member

@szha szha left a comment

Choose a reason for hiding this comment

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

Thank you for fixing it.

@szha szha merged commit 25ef9bb into apache:master Nov 15, 2017
@JulianSlzr JulianSlzr deleted the hybrid-lambda-repr branch November 15, 2017 17:08
eric-haibin-lin pushed a commit to eric-haibin-lin/mxnet that referenced this pull request Dec 3, 2017
* fix HybridLambda repr

* ensure function name printed by HybridBlock repr when string
JulianSlzr added a commit to JulianSlzr/incubator-mxnet that referenced this pull request Dec 5, 2017
JulianSlzr added a commit to JulianSlzr/incubator-mxnet that referenced this pull request Dec 6, 2017
JulianSlzr added a commit to JulianSlzr/incubator-mxnet that referenced this pull request Dec 8, 2017
piiswrong pushed a commit that referenced this pull request Dec 9, 2017
* Fix __repr__ for gluon.parameter

* Add to contributors (PRs #8565, #8956, etc.)

* Add unit test for gluon.Parameter string
cjolivier01 pushed a commit to cjolivier01/mxnet that referenced this pull request Dec 10, 2017
* Fix __repr__ for gluon.parameter

* Add to contributors (PRs apache#8565, apache#8956, etc.)

* Add unit test for gluon.Parameter string
szha pushed a commit that referenced this pull request Dec 14, 2017
* Fix __repr__ for gluon.parameter

* Add to contributors (PRs #8565, #8956, etc.)

* Add unit test for gluon.Parameter string
rahul003 pushed a commit to rahul003/mxnet that referenced this pull request Jun 4, 2018
* fix HybridLambda repr

* ensure function name printed by HybridBlock repr when string
rahul003 pushed a commit to rahul003/mxnet that referenced this pull request Jun 4, 2018
* Fix __repr__ for gluon.parameter

* Add to contributors (PRs apache#8565, apache#8956, etc.)

* Add unit test for gluon.Parameter string
zheng-da pushed a commit to zheng-da/incubator-mxnet that referenced this pull request Jun 28, 2018
* Fix __repr__ for gluon.parameter

* Add to contributors (PRs apache#8565, apache#8956, etc.)

* Add unit test for gluon.Parameter string
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants