dmlc / gluon-nlp

NLP made easy
https://nlp.gluon.ai/
Apache License 2.0
2.55k stars 538 forks source link

[Numpy][Tiny Fix]Some fixes to make the CI more stable #1265

Closed sxjscience closed 4 years ago

sxjscience commented 4 years ago

Adds several fixes to make the CI pass and more stable. I find that XLMR seems to be too large for Github action so I disabled the forward test.

@leezu

codecov[bot] commented 4 years ago

Codecov Report

Merging #1265 into numpy will increase coverage by 0.07%. The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            numpy    #1265      +/-   ##
==========================================
+ Coverage   82.45%   82.52%   +0.07%     
==========================================
  Files          38       38              
  Lines        5488     5500      +12     
==========================================
+ Hits         4525     4539      +14     
+ Misses        963      961       -2     
Impacted Files Coverage Δ
src/gluonnlp/models/__init__.py 96.55% <ø> (ø)
src/gluonnlp/utils/lazy_imports.py 56.57% <66.66%> (+0.86%) :arrow_up:
src/gluonnlp/data/tokenizers.py 77.71% <76.47%> (-0.07%) :arrow_down:
src/gluonnlp/data/loading.py 83.39% <0.00%> (+2.26%) :arrow_up: