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

Conversation

crcrpar
Copy link
Contributor

@crcrpar crcrpar commented Apr 27, 2019

Description

This fixes nlp.data.WikiText2(segment='train')[0][0:5] to nlp.data.WikiText2(segment='train')[0:5] because nlp.data.WikiText2(segment='train')[0][0:5] is '='.

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

Comments

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

@crcrpar crcrpar requested a review from szha as a code owner April 27, 2019 01:47
@crcrpar crcrpar changed the title [TUTORIAL [TUTORIAL] Fix README WikiText2 example Apr 27, 2019
Copy link
Member

@eric-haibin-lin eric-haibin-lin left a comment

Choose a reason for hiding this comment

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

thanks for the contribution!

@eric-haibin-lin
Copy link
Member

@szha is the CI down?

@szha
Copy link
Member

szha commented Apr 29, 2019

@eric-haibin-lin a recent kernel image upgrade on the CI server caused nvidia drivers to fail. I reverted the kernel images just now so hopefully, things should be working now

@codecov
Copy link

codecov bot commented May 1, 2019

Codecov Report

Merging #681 into master will decrease coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #681      +/-   ##
==========================================
- Coverage   65.27%   65.03%   -0.25%     
==========================================
  Files         143      143              
  Lines       13279    13117     -162     
==========================================
- Hits         8668     8530     -138     
+ Misses       4611     4587      -24
Flag Coverage Δ
#PR663 ?
#PR681 65.03% <ø> (?)
#master ?
#notserial 41.91% <ø> (+0.49%) ⬆️
#py2 64.78% <ø> (-0.09%) ⬇️
#py3 64.91% <ø> (-0.25%) ⬇️
#serial 50.78% <ø> (-0.85%) ⬇️

1 similar comment
@codecov
Copy link

codecov bot commented May 1, 2019

Codecov Report

Merging #681 into master will decrease coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #681      +/-   ##
==========================================
- Coverage   65.27%   65.03%   -0.25%     
==========================================
  Files         143      143              
  Lines       13279    13117     -162     
==========================================
- Hits         8668     8530     -138     
+ Misses       4611     4587      -24
Flag Coverage Δ
#PR663 ?
#PR681 65.03% <ø> (?)
#master ?
#notserial 41.91% <ø> (+0.49%) ⬆️
#py2 64.78% <ø> (-0.09%) ⬇️
#py3 64.91% <ø> (-0.25%) ⬇️
#serial 50.78% <ø> (-0.85%) ⬇️

@mli
Copy link
Member

mli commented May 2, 2019

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

@szha szha merged commit e867709 into dmlc:master May 2, 2019
@szha
Copy link
Member

szha commented May 2, 2019

@crcrpar thank you for the fix!

@crcrpar
Copy link
Contributor Author

crcrpar commented May 2, 2019

My pleasure!

@crcrpar crcrpar deleted the fix-readme-example branch May 2, 2019 08:42
gigasquid pushed a commit to gigasquid/gluon-nlp that referenced this pull request May 6, 2019
astonzhang pushed a commit that referenced this pull request May 10, 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