Closes #275 | Create dataset loader for UIT-ViCoV19QA #275

[Gyyz]

Closes #275

Checkbox

• [x] Confirm that this PR is linked to the dataset issue.
• [x] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
• [x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
• [x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
• [x] 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.
• [x] Confirm dataloader script works with datasets.load_dataset function.
• [x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py.
• [ ] 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.

[Gyyz]

Scripts Passed.

1. subset = uitvicov19qa1/2/3/4_ans: python -m tests.test_seacrowd seacrowd/sea_datasets/uit_vicov19qa/uit_vicov19qa.py --subset uit_vicov19qa_4_ans

2. make check_file=seacrowd/sea_datasets/uit_vicov19qa/uit_vicov19qa.py

[Gyyz]

Checked, LGTM.

One question: Is there any reasons why we have 4 subsets? Maybe we can just unite them into one, they are similar to me. What do you think @raileymontalan @Gyyz ?

I am OK with merging to just one

[raileymontalan]

Checked, LGTM.

One question: Is there any reasons why we have 4 subsets? Maybe we can just unite them into one, they are similar to me. What do you think @raileymontalan @Gyyz ?

Agree. @Gyyz please implement only one subset here.

[Gyyz]

Checked, LGTM. One question: Is there any reasons why we have 4 subsets? Maybe we can just unite them into one, they are similar to me. What do you think @raileymontalan @Gyyz ?

Agree. @Gyyz please implement only one subset here.

@MJonibek @raileymontalan, hello, updated, please check. Scripts Passed: 1.python -m tests.test_seacrowd seacrowd/sea_datasets/uit_vicov19qa/uit_vicov19qa.py 2.make check_file=seacrowd/sea_datasets/uit_vicov19qa/uit_vicov19qa.py

[Gyyz]

Checked, LGTM! One small nit: can you rerun make check_file=seacrowd/sea_datasets/uit_vicov19qa/uit_vicov19qa.py command, I see some changes after running it:

@MJonibek Yes, after make script, the dictionary structure of the sample will be changed, and all keys and values will be placed on the same line, which affects the code's readability a bit. So I will adjust the code again on the example dict after make script