asserts replaced by exception for text classification task with test.
MEMBER
Haha it looks like you got the chance of being reviewed twice at the same time and got the same suggestion twice x)
Anyway it's all good now so we can merge !
2021-11-12T14:56:53Z
https://github.com/huggingface/datasets/pull/3256#issuecomment-967179375
https://api.github.com/repos/huggingface/datasets/issues/3256
IC_kwDODunzps45pfxv
https://api.github.com/repos/huggingface/datasets/issues/comments/967179375/reactions
2021-11-12T14:56:53Z
https://api.github.com/repos/huggingface/datasets/issues/comments/967179375
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
CONTRIBUTOR
Thanks for the feedback. 
2021-11-12T15:09:33Z
https://github.com/huggingface/datasets/pull/3256#issuecomment-967189727
https://api.github.com/repos/huggingface/datasets/issues/3256
IC_kwDODunzps45piTf
https://api.github.com/repos/huggingface/datasets/issues/comments/967189727/reactions
2021-11-12T15:09:33Z
https://api.github.com/repos/huggingface/datasets/issues/comments/967189727
https://avatars.githubusercontent.com/u/153142?v=4
https://api.github.com/users/manisnesan/events{/privacy}
https://api.github.com/users/manisnesan/followers
https://api.github.com/users/manisnesan/following{/other_user}
https://api.github.com/users/manisnesan/gists{/gist_id}
https://github.com/manisnesan
manisnesan
MDQ6VXNlcjE1MzE0Mg==
https://api.github.com/users/manisnesan/orgs
https://api.github.com/users/manisnesan/received_events
https://api.github.com/users/manisnesan/repos
https://api.github.com/users/manisnesan/starred{/owner}{/repo}
https://api.github.com/users/manisnesan/subscriptions
User
https://api.github.com/users/manisnesan
public
I have replaced only a single assert in text_classification.py along with a unit test to verify an exception is raised based on https://github.com/huggingface/datasets/issues/3171 . 

I would like to first understand the code contribution workflow. So keeping the change to a single file rather than making too many changes. Once this gets approved, I will look into the rest. 

Thanks. 