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

Conversation

rongruosong
Copy link
Contributor

@rongruosong rongruosong commented Apr 8, 2019

Description

(Brief description on what this PR is about)

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

@rongruosong rongruosong requested a review from szha as a code owner April 8, 2019 13:12
@mli
Copy link
Member

mli commented Apr 8, 2019

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

@codecov
Copy link

codecov bot commented Apr 8, 2019

Codecov Report

Merging #656 into master will increase coverage by 0.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #656      +/-   ##
==========================================
+ Coverage   63.04%   63.73%   +0.68%     
==========================================
  Files         151      141      -10     
  Lines       13779    12902     -877     
==========================================
- Hits         8687     8223     -464     
+ Misses       5092     4679     -413
Flag Coverage Δ
#PR612 ?
#PR656 63.73% <ø> (?)
#master ?
#notserial 42.62% <ø> (+2.73%) ⬆️
#py2 63.49% <ø> (-0.37%) ⬇️
#py3 63.61% <ø> (+0.68%) ⬆️
#serial 49.19% <ø> (-0.22%) ⬇️

1 similar comment
@codecov
Copy link

codecov bot commented Apr 8, 2019

Codecov Report

Merging #656 into master will increase coverage by 0.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #656      +/-   ##
==========================================
+ Coverage   63.04%   63.73%   +0.68%     
==========================================
  Files         151      141      -10     
  Lines       13779    12902     -877     
==========================================
- Hits         8687     8223     -464     
+ Misses       5092     4679     -413
Flag Coverage Δ
#PR612 ?
#PR656 63.73% <ø> (?)
#master ?
#notserial 42.62% <ø> (+2.73%) ⬆️
#py2 63.49% <ø> (-0.37%) ⬇️
#py3 63.61% <ø> (+0.68%) ⬆️
#serial 49.19% <ø> (-0.22%) ⬇️

@szha
Copy link
Member

szha commented Apr 8, 2019

@rongruosong good catch. thank you for the fix

@astonzhang astonzhang self-requested a review April 8, 2019 19:04
Copy link
Member

@astonzhang astonzhang left a comment

Choose a reason for hiding this comment

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

Thanks!

@astonzhang astonzhang merged commit b515c23 into dmlc:master Apr 8, 2019
gigasquid pushed a commit to gigasquid/gluon-nlp that referenced this pull request May 6, 2019
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.

4 participants