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

[DOC] add missing docs for loss #814

Merged
merged 1 commit into from
Jul 5, 2019
Merged

[DOC] add missing docs for loss #814

merged 1 commit into from
Jul 5, 2019

Conversation

szha
Copy link
Member

@szha szha commented Jul 4, 2019

Description

add missing docs for loss

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

  • add missing docs for MaskedSoftmaxCrossEntropyLoss and LabelSmoothing

@mli
Copy link
Member

mli commented Jul 5, 2019

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

@szha szha requested a review from a team July 5, 2019 05:54
@eric-haibin-lin eric-haibin-lin merged commit f7b830d into dmlc:master Jul 5, 2019
@szha szha deleted the doc branch July 5, 2019 18:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@leezu leezu leezu approved these changes

@eric-haibin-lin eric-haibin-lin eric-haibin-lin approved these changes

@TaoLv TaoLv TaoLv approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@szha @mli @leezu @eric-haibin-lin @TaoLv