Skip to content

Final #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Sep 30, 2018
Merged

Final #3

merged 5 commits into from
Sep 30, 2018

Conversation

xuyige
Copy link
Collaborator

@xuyige xuyige commented Sep 30, 2018

No description provided.

FengZiYjun and others added 5 commits September 23, 2018 16:03
- add character vocab in preprocessor
- add dataset loader for language model dataset
- other minor adjustments
- preserve only a little example data for language model
@codecov-io
Copy link

Codecov Report

Merging #3 into master will increase coverage by 0.52%.
The diff coverage is 79.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   73.11%   73.64%   +0.52%     
==========================================
  Files          67       67              
  Lines        3437     3468      +31     
==========================================
+ Hits         2513     2554      +41     
+ Misses        924      914      -10
Impacted Files Coverage Δ
fastNLP/models/char_language_model.py 0% <0%> (ø) ⬆️
fastNLP/modules/aggregator/self_attention.py 0% <0%> (ø) ⬆️
test/model/test_cws.py 96.82% <100%> (+0.15%) ⬆️
test/core/test_tester.py 100% <100%> (ø) ⬆️
test/model/test_seq_label.py 100% <100%> (ø) ⬆️
fastNLP/core/predictor.py 94.33% <100%> (-1.05%) ⬇️
fastNLP/core/tester.py 84.61% <100%> (+6.92%) ⬆️
test/saver/test_config_saver.py 100% <100%> (ø) ⬆️
fastNLP/loader/base_loader.py 70% <100%> (+12.85%) ⬆️
fastNLP/core/batch.py 97.05% <100%> (+1.93%) ⬆️
... and 28 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 28a0683...91f3d97. Read the comment docs.

@FengZiYjun FengZiYjun merged commit 1d4e406 into FengZiYjun:master Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants