fastnlp / fastNLP

fastNLP: A Modularized and Extensible NLP Framework. Currently still in incubation.
https://gitee.com/fastnlp/fastNLP
Apache License 2.0
3.06k stars 450 forks source link

出现非法格式catch住的时候,应该将sample清空 #264

Closed Gosicfly closed 4 years ago

Gosicfly commented 4 years ago

出现非法格式catch住的时候,应该将sample清空,不然会一直报warning,sample也会一直累积新的内容得不到正确的更新。

Description:简要描述这次PR的内容

Main reason: 做出这次修改的原因

Checklist 检查下面各项是否完成

Please feel free to remove inapplicable items for your PR.

Changes: 逐项描述修改的内容

Mention: 找人review你的PR

@修改过这个文件的人 @核心开发人员

codecov-io commented 4 years ago

Codecov Report

Merging #264 into master will increase coverage by 0.01%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #264      +/-   ##
==========================================
+ Coverage   77.42%   77.43%   +0.01%     
==========================================
  Files          84       84              
  Lines       10145    10146       +1     
==========================================
+ Hits         7855     7857       +2     
+ Misses       2290     2289       -1
Impacted Files Coverage Δ
fastNLP/io/file_reader.py 70.73% <0%> (-0.88%) :arrow_down:
fastNLP/core/callback.py 68.73% <0%> (+0.35%) :arrow_up:

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 03e9ca2...baf009f. Read the comment docs.