-
Notifications
You must be signed in to change notification settings - Fork 538
[DATASET] add LCQMC, ChnSentiCorp dataset #774
[DATASET] add LCQMC, ChnSentiCorp dataset #774
Conversation
Codecov Report
|
Codecov Report
@@ Coverage Diff @@
## master #774 +/- ##
==========================================
+ Coverage 90.21% 91.09% +0.88%
==========================================
Files 64 64
Lines 6233 6469 +236
==========================================
+ Hits 5623 5893 +270
+ Misses 610 576 -34
|
Job PR-774/1 is complete. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! What are the licenses of these datasets? Could you add them to the classes?
If the dataset licenses are friendly, we can consider redistributing them.
OK, I can add the license for LCQMC and XNLI. |
We probably can't redistribute XNLI due to the non-commercial license. |
Job PR-774/2 is complete. |
Job PR-774/3 is complete. |
Oh. What a pity. |
It only means that we cannot host the dataset, right? Using external links is fine? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you fix the lint failure ?
yes that's what I meant |
Job PR-774/4 is complete. |
Job PR-774/5 is complete. |
Job PR-774/6 is complete. |
Description
add LCQMC, ChnSentiCorp dataset
Checklist
Essentials
Changes
Comments