Closes #223 | Add/Update Dataloader VinText

[patrickamadeus]

Closes #223

Checkbox

• Confirm that this PR is linked to the dataset issue.
• Create the dataloader script seacrowd/sea_datasets/{my_dataset}/{my_dataset}.py (please use only lowercase and underscore for dataset folder naming, as mentioned in dataset issue) and its __init__.py within {my_dataset} folder.
• Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _LOCAL, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
• Implement _info(), _split_generators() and _generate_examples() in dataloader script.
• Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
• Confirm dataloader script works with datasets.load_dataset function.
• Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py or python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py --subset_id {subset_name_without_source_or_seacrowd_suffix}.
• [.] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Tests

[akhdanfadh]

Dataloader tested and worked at first, but please address my discussions below.

[patrickamadeus]

Hi @akhdanfadh! I've addressed the suggestions! For now I will change the description and proceed with the original version of dataset.

Thank you for the meaningful reviews!

[sabilmakbar]

Hi, @akhdanfadh @fhudi. Would you like to review the latest code changes? Thanks!

[holylovenia]

Hi @fhudi @akhdanfadh, I would like to let you know that we plan to finalize the calculation of the open contributions (e.g., dataloader implementations) in 31 hours, so it'd be great if we could wrap up the reviewing and merge this PR before then.

cc: @patrickamadeus

[fhudi]

LGTM.
Passed the tests.
Thanks for being cooperative!

[fhudi]

Hi @akhdanfadh,
It seems all the previous concerns have been addressed.
Could you please re-review the latest code changes? thanks.
cc: @sabilmakbar

[akhdanfadh]

@fhudi The code tested and OK. My only comment is that the change in image_text.py schema is insignificant. Could we remove that?

[fhudi]

@akhdanfadh sure, please do.
I am fine with either one.

[akhdanfadh]

Merging now. @patrickamadeus @fhudi

[fhudi]

Thanks for adding additional comment here.
Though, too bad that the new line at the EOF is gone 🥲