Closes #533 | Add Dataloader XED

muhammadravi251001 commented 4 months ago

Title: Add Dataloader XED

First line PR Message: Closes https://github.com/SEACrowd/seacrowd-datahub/issues/533

Note:

I changed the _LICENSE by Licenses.CC_BY_4_0.value despite of license information on the dataloader issue here https://github.com/SEACrowd/seacrowd-datahub/issues/533 is Licenses.UNKNOWN.value. I'd rather take license information from the bottom of homepage's README.md.
I also changed the _SUPPORTED_TASKS to Tasks.ASPECT_BASED_SENTIMENT_ANALYSIS instead of Tasks.SENTIMENT_ANALYSIS due to the fact that this dataset is multi-label, so I will need text_multi as schema not text.
From the homepage, I found that Indonesian is already there (it's already told on this comment: https://github.com/SEACrowd/seacrowd-datahub/issues/533#issuecomment-2003237075). So, I add the Indonesian dataloader too for this dataset --> I changed the _LANGUAGES to ["ind", "vie"].

[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 folder naming, as mentioned in dataset issue) and its __init__.py within {my_dataset} folder.
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _LOCAL, _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 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.

muhammadravi251001 commented 4 months ago

LGTM!

Thanks for the approval, Sir!

muhammadravi251001 commented 4 months ago

It worked perfectly well. LGTM

Thanks for the approval, Sir!

ljvmiranda921 commented 4 months ago

Merging in a few :)

SEACrowd / seacrowd-datahub