Closes #193 | Create dataset loader for MALINDO Morph

[danjohnvelasco]

Closes #193

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.
• [ ] 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.

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.

This dataset is a lexicon. It does not support any tasks. This case is similar to https://github.com/SEACrowd/seacrowd-datahub/pull/155

To run tests:

python -m tests.test_seacrowd_source_only seacrowd/sea_datasets/malindo_morph/malindo_morph.py

[danjohnvelasco]

Other than one minor fix below, I have nothing more to suggest. LGTM!

Fixed, thanks for the review! :)

[holylovenia]

Replacing @gentaiscool with @jen-santoso due to no response.

[danjohnvelasco]

Hi @danjohnvelasco, thank you for the dataloader implementation.

So this case is a source-only, right? The dataloader runs well, just two nits here and the dataloader's good to go!

Hi @jen-santoso, thanks for the review. Yes, this is source-only. Your suggested changes are commited.

[jensan-1]

OK LGTM, merging now.