Replace print with logging in dataset scripts
COLLABORATOR
@lhoestq Maybe a script or even a test in `test_dataset_common.py` that verifies that a dataset script meets some set of quality standards (print calls and todos from the dataset script template are not present, etc.) could be added?
2021-03-12T02:44:04Z
https://github.com/huggingface/datasets/pull/2019#issuecomment-797195326
https://api.github.com/repos/huggingface/datasets/issues/2019
MDEyOklzc3VlQ29tbWVudDc5NzE5NTMyNg==
https://api.github.com/repos/huggingface/datasets/issues/comments/797195326/reactions
2021-03-12T02:44:26Z
https://api.github.com/repos/huggingface/datasets/issues/comments/797195326
https://avatars.githubusercontent.com/u/47462742?v=4
https://api.github.com/users/mariosasko/events{/privacy}
https://api.github.com/users/mariosasko/followers
https://api.github.com/users/mariosasko/following{/other_user}
https://api.github.com/users/mariosasko/gists{/gist_id}
https://github.com/mariosasko
mariosasko
MDQ6VXNlcjQ3NDYyNzQy
https://api.github.com/users/mariosasko/orgs
https://api.github.com/users/mariosasko/received_events
https://api.github.com/users/mariosasko/repos
https://api.github.com/users/mariosasko/starred{/owner}{/repo}
https://api.github.com/users/mariosasko/subscriptions
User
https://api.github.com/users/mariosasko
public
MEMBER
Yes definitely !
2021-03-12T10:09:01Z
https://github.com/huggingface/datasets/pull/2019#issuecomment-797384055
https://api.github.com/repos/huggingface/datasets/issues/2019
MDEyOklzc3VlQ29tbWVudDc5NzM4NDA1NQ==
https://api.github.com/repos/huggingface/datasets/issues/comments/797384055/reactions
2021-03-12T10:09:01Z
https://api.github.com/repos/huggingface/datasets/issues/comments/797384055
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
Replaces `print(...)` in the dataset scripts with the library logger.