SUMIN/HuggingFace_transformer
1
0
Fork 0
Code Issues Pull Requests Actions 7 Packages Projects Releases Wiki Activity

Fix and re-enable ConversationalPipeline tests (#26907)

Browse Source 
* Fix and re-enable conversationalpipeline tests

* Fix the batch test so the change only applies to conversational pipeline
This commit is contained in:
Matt
2023-10-19 12:04:25 +01:00
committed by GitHub
parent 734dd96e02
commit bdbcd5d482
2 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
tests/pipelines/test_pipelines_conversational.py
View File

@@ -77,14 +77,14 @@ class ConversationalPipelineTests(unittest.TestCase):
def run_pipeline_test(self, conversation_agent, _): def run_pipeline_test(self, conversation_agent, _):
# Simple # Simple
outputs = conversation_agent(Conversation("Hi there!")) outputs = conversation_agent(Conversation("Hi there!"), max_new_tokens=20)
self.assertEqual( self.assertEqual(
outputs, outputs,
Conversation([{"role": "user", "content": "Hi there!"}, {"role": "assistant", "content": ANY(str)}]), Conversation([{"role": "user", "content": "Hi there!"}, {"role": "assistant", "content": ANY(str)}]),
) )
# Single list # Single list
outputs = conversation_agent([Conversation("Hi there!")]) outputs = conversation_agent([Conversation("Hi there!")], max_new_tokens=20)
self.assertEqual( self.assertEqual(
outputs, outputs,
Conversation([{"role": "user", "content": "Hi there!"}, {"role": "assistant", "content": ANY(str)}]), Conversation([{"role": "user", "content": "Hi there!"}, {"role": "assistant", "content": ANY(str)}]),
@@ -96,7 +96,7 @@ class ConversationalPipelineTests(unittest.TestCase):
self.assertEqual(len(conversation_1), 1) self.assertEqual(len(conversation_1), 1)
self.assertEqual(len(conversation_2), 1) self.assertEqual(len(conversation_2), 1)
outputs = conversation_agent([conversation_1, conversation_2]) outputs = conversation_agent([conversation_1, conversation_2], max_new_tokens=20)
self.assertEqual(outputs, [conversation_1, conversation_2]) self.assertEqual(outputs, [conversation_1, conversation_2])
self.assertEqual( self.assertEqual(
outputs, outputs,
@@ -118,7 +118,7 @@ class ConversationalPipelineTests(unittest.TestCase):
# One conversation with history # One conversation with history
conversation_2.add_message({"role": "user", "content": "Why do you recommend it?"}) conversation_2.add_message({"role": "user", "content": "Why do you recommend it?"})
outputs = conversation_agent(conversation_2) outputs = conversation_agent(conversation_2, max_new_tokens=20)
self.assertEqual(outputs, conversation_2) self.assertEqual(outputs, conversation_2)
self.assertEqual( self.assertEqual(
outputs, outputs,

5
tests/test_pipeline_mixin.py
View File

@@ -312,6 +312,10 @@ class PipelineTesterMixin:
yield copy.deepcopy(random.choice(examples)) yield copy.deepcopy(random.choice(examples))
out = [] out = []
if task == "conversational":
for item in pipeline(data(10), batch_size=4, max_new_tokens=20):
out.append(item)
else:
for item in pipeline(data(10), batch_size=4): for item in pipeline(data(10), batch_size=4):
out.append(item) out.append(item)
self.assertEqual(len(out), 10) self.assertEqual(len(out), 10)
@@ -327,7 +331,6 @@ class PipelineTesterMixin:
self.run_task_tests(task="automatic-speech-recognition") self.run_task_tests(task="automatic-speech-recognition")
@is_pipeline_test @is_pipeline_test
@unittest.skip("Conversational tests are currently broken for several models, will fix ASAP - Matt")
def test_pipeline_conversational(self): def test_pipeline_conversational(self):
self.run_task_tests(task="conversational") self.run_task_tests(task="conversational")