Fix NonMatchingChecksumError in mbpp dataset
NONE
_The documentation is not available anymore as the PR was closed or merged._
2022-08-04T08:25:06Z
https://github.com/huggingface/datasets/pull/4788#issuecomment-1204930907
https://api.github.com/repos/huggingface/datasets/issues/4788
IC_kwDODunzps5H0clb
https://api.github.com/repos/huggingface/datasets/issues/comments/1204930907/reactions
2022-08-04T17:34:00Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1204930907
https://avatars.githubusercontent.com/u/99929124?v=4
https://api.github.com/users/HuggingFaceDocBuilderDev/events{/privacy}
https://api.github.com/users/HuggingFaceDocBuilderDev/followers
https://api.github.com/users/HuggingFaceDocBuilderDev/following{/other_user}
https://api.github.com/users/HuggingFaceDocBuilderDev/gists{/gist_id}
https://github.com/HuggingFaceDocBuilderDev
HuggingFaceDocBuilderDev
U_kgDOBfTMJA
https://api.github.com/users/HuggingFaceDocBuilderDev/orgs
https://api.github.com/users/HuggingFaceDocBuilderDev/received_events
https://api.github.com/users/HuggingFaceDocBuilderDev/repos
https://api.github.com/users/HuggingFaceDocBuilderDev/starred{/owner}{/repo}
https://api.github.com/users/HuggingFaceDocBuilderDev/subscriptions
User
https://api.github.com/users/HuggingFaceDocBuilderDev
public
NONE
Thank you for the quick response! Before noticing that you already had implemented the fix, I already had implemened my own version. I'd also suggest bumping the major version because the contents of the dataset changed, even if only slightly.
I'll attach my version of the affected files: [mbpp-checksum-changes.zip](https://github.com/huggingface/datasets/files/9258161/mbpp-checksum-changes.zip).
2022-08-04T08:54:50Z
https://github.com/huggingface/datasets/pull/4788#issuecomment-1204964085
https://api.github.com/repos/huggingface/datasets/issues/4788
IC_kwDODunzps5H0kr1
https://api.github.com/repos/huggingface/datasets/issues/comments/1204964085/reactions
2022-08-04T08:55:02Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1204964085
https://avatars.githubusercontent.com/u/2452384?v=4
https://api.github.com/users/stadlerb/events{/privacy}
https://api.github.com/users/stadlerb/followers
https://api.github.com/users/stadlerb/following{/other_user}
https://api.github.com/users/stadlerb/gists{/gist_id}
https://github.com/stadlerb
stadlerb
MDQ6VXNlcjI0NTIzODQ=
https://api.github.com/users/stadlerb/orgs
https://api.github.com/users/stadlerb/received_events
https://api.github.com/users/stadlerb/repos
https://api.github.com/users/stadlerb/starred{/owner}{/repo}
https://api.github.com/users/stadlerb/subscriptions
User
https://api.github.com/users/stadlerb
public
MEMBER
Hi @stadlerb, thanks for your feedback.

We normally update the major version whenever there is a new dataset release, usually with a breaking change in schema. The patch version is updated whenever there is a small correction in the dataset that does not change its schema.

As a side note for future contributions, please note that this dataset is hosted in our library GitHub repository. Therefore, the PRs to GitHub-hosted datasets needs being done through GitHub.

Currently added datasets are hosted on the Hub and for them, PRs can be done through the Hub.
2022-08-04T09:26:46Z
https://github.com/huggingface/datasets/pull/4788#issuecomment-1204999188
https://api.github.com/repos/huggingface/datasets/issues/4788
IC_kwDODunzps5H0tQU
https://api.github.com/repos/huggingface/datasets/issues/comments/1204999188/reactions
2022-08-04T09:28:29Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1204999188
https://avatars.githubusercontent.com/u/8515462?v=4
https://api.github.com/users/albertvillanova/events{/privacy}
https://api.github.com/users/albertvillanova/followers
https://api.github.com/users/albertvillanova/following{/other_user}
https://api.github.com/users/albertvillanova/gists{/gist_id}
https://github.com/albertvillanova
albertvillanova
MDQ6VXNlcjg1MTU0NjI=
https://api.github.com/users/albertvillanova/orgs
https://api.github.com/users/albertvillanova/received_events
https://api.github.com/users/albertvillanova/repos
https://api.github.com/users/albertvillanova/starred{/owner}{/repo}
https://api.github.com/users/albertvillanova/subscriptions
User
https://api.github.com/users/albertvillanova
public
NONE
I just noticed another problem with the dataset: The [GitHub page](https://github.com/google-research/google-research/tree/master/mbpp) and the [paper](http://arxiv.org/abs/2108.07732) mention a train-test split, which is not reflected in the dataloader. I'll open a new issue regarding this later.
2022-08-04T14:53:05Z
https://github.com/huggingface/datasets/pull/4788#issuecomment-1205367994
https://api.github.com/repos/huggingface/datasets/issues/4788
IC_kwDODunzps5H2HS6
https://api.github.com/repos/huggingface/datasets/issues/comments/1205367994/reactions
2022-08-04T14:53:05Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1205367994
https://avatars.githubusercontent.com/u/2452384?v=4
https://api.github.com/users/stadlerb/events{/privacy}
https://api.github.com/users/stadlerb/followers
https://api.github.com/users/stadlerb/following{/other_user}
https://api.github.com/users/stadlerb/gists{/gist_id}
https://github.com/stadlerb
stadlerb
MDQ6VXNlcjI0NTIzODQ=
https://api.github.com/users/stadlerb/orgs
https://api.github.com/users/stadlerb/received_events
https://api.github.com/users/stadlerb/repos
https://api.github.com/users/stadlerb/starred{/owner}{/repo}
https://api.github.com/users/stadlerb/subscriptions
User
https://api.github.com/users/stadlerb
public
Fix issue reported on the Hub: https://huggingface.co/datasets/mbpp/discussions/1

Fix #4787. 