Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

[FEATURE] Local Adam for Transformer #868

Open
wants to merge 9 commits into
base: v0.x
Choose a base branch
from

Conversation

xcgoner
Copy link

@xcgoner xcgoner commented Aug 7, 2019

Description

Add local Adam trainer for Transformer

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

@xcgoner xcgoner requested a review from szha as a code owner August 7, 2019 23:18
@mli
Copy link
Member

mli commented Aug 7, 2019

Job PR-868/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-868/1/index.html

@mli
Copy link
Member

mli commented Aug 8, 2019

Job PR-868/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-868/2/index.html

@szha szha changed the base branch from master to v0.x August 13, 2020 02:18
@szha szha requested a review from a team as a code owner August 13, 2020 02:18
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