Updating the tests after the conversation teacher changes.

facebookresearch / ParlAI

A framework for training and evaluating AI models on a variety of openly available dialogue datasets.

https://parl.ai

MIT License

10.49k stars 2.1k forks source link

Updating the tests after the conversation teacher changes. #5038

Closed mojtaba-komeili closed 1 year ago

mojtaba-komeili commented 1 year ago

Patch description Updated the teacher tests for reflecting the changes from https://github.com/facebookresearch/ParlAI/pull/5033.

mojtaba-komeili commented 1 year ago

@klshuster There is some issue with the LCCC task and yaml. Given that it is not one of our core tasks does it make sense to relax its tests?

mojtaba-komeili commented 1 year ago

@Zheng0428 based on your submission, are you able to fix this issue with this dataset? In summary:

self = <pytest_regressions.data_regression.RegressionYamlDumper object at 0x7f0c90969ca0>
data = {'id': 'partner1', 'text': '我饿了。'}

    def represent_undefined(self, data):
>       raise RepresenterError("cannot represent an object", data)
E       yaml.representer.RepresenterError: ('cannot represent an object', {'id': 'partner1', 'text': '我饿了。'})