search

cardiffnlp / xlm-t

Repository for XLM-T, a framework for evaluating multilingual language models on Twitter data
Apache License 2.0
142 stars 23 forks source link

Potential (very minor) bug in fine-tuning script #5

Open micahcarroll opened 2 years ago

micahcarroll commented 2 years ago

I believe this line may be a bug, as it overrides the global variable set earlier in the file:

https://github.com/cardiffnlp/xlm-t/blob/aa9b15ef035876cec53d4ce55c04ec2e93acb92d/src/adapter_finetuning.py#L63

AADeLucia commented 2 years ago

I also saw this, I fixed it in my copy of the repo by deleting that line and changing 

dataset_dict = fetch_data(language, files)

to 

dataset_dict = fetch_data(LANGUAGE, files)