dmlc / gluon-nlp

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

Add classification script #1517

Closed xinyual closed 3 years ago

xinyual commented 3 years ago

Description

(Brief description on what this PR is about)

Checklist

Essentials

Changes

Comments

cc @dmlc/gluon-nlp-team

github-actions[bot] commented 3 years ago

The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1517/bb1a93b55baa8d7d3cde27faafeae9778d7cd08c/index.html

codecov[bot] commented 3 years ago

Codecov Report

Merging #1517 (807073b) into master (601c752) will decrease coverage by 0.47%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1517      +/-   ##
==========================================
- Coverage   86.34%   85.87%   -0.48%     
==========================================
  Files          55       55              
  Lines        7513     7396     -117     
==========================================
- Hits         6487     6351     -136     
- Misses       1026     1045      +19     
Impacted Files Coverage Δ
conftest.py 76.31% <0.00%> (-9.94%) :arrow_down:
src/gluonnlp/data/loading.py 75.75% <0.00%> (-2.36%) :arrow_down:
src/gluonnlp/utils/lazy_imports.py 58.42% <0.00%> (-2.25%) :arrow_down:
src/gluonnlp/utils/misc.py 54.86% <0.00%> (-1.50%) :arrow_down:
src/gluonnlp/data/tokenizers/spacy.py 65.33% <0.00%> (-0.91%) :arrow_down:
src/gluonnlp/data/tokenizers/huggingface.py 71.06% <0.00%> (-0.78%) :arrow_down:
src/gluonnlp/data/tokenizers/jieba.py 73.13% <0.00%> (-0.40%) :arrow_down:
src/gluonnlp/models/transformer_xl.py 80.48% <0.00%> (-0.39%) :arrow_down:
src/gluonnlp/models/xlmr.py 87.87% <0.00%> (-0.36%) :arrow_down:
src/gluonnlp/models/electra.py 76.50% <0.00%> (-0.34%) :arrow_down:
... and 17 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 601c752...807073b. Read the comment docs.

github-actions[bot] commented 3 years ago

The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1517/8968798bb2e26a164e3b50f40651d842859c70d4/index.html

sxjscience commented 3 years ago

Would you support tensorboard?

github-actions[bot] commented 3 years ago

The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1517/2419e898bf39dbd944d93f41862ca58a951bc703/index.html

leezu commented 3 years ago

Could you fix this warning in the documentation build so that the CI will pass

[2021-03-05T00:43:22.211Z] /workspace/gluon-nlp/docs/model_zoo/classification/index.rst: WARNING: document isn't included in any toctree

Essentially you need to link the file in https://github.com/dmlc/gluon-nlp/blob/master/scripts/index.rst

leezu commented 3 years ago

The issue [2021-03-10T11:39:50.280Z] /workspace/gluon-nlp/docs/model_zoo/classification/index.rst: WARNING: document isn't included in any toctree still remains

leezu commented 3 years ago

@barry-jin do you think the issue is related to the CI website-build setup?

barry-jin commented 3 years ago

@barry-jin do you think the issue is related to the CI website-build setup? Try to add classification/index in toctree


.. toctree::
:hidden:
:maxdepth: 1

benchmarks/index

classification/index conversion_toolkits/index datasets/index generation/index machine_translation/index pretraining/index processing/index question_answering/index

github-actions[bot] commented 3 years ago

The documentation website for preview: http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR1517/807073b75d407c47185628611725160a799a6ed4/index.html