Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Fix variable name in model chat test #3837

Merged
merged 4 commits into from Jul 23, 2021
Merged

Fix variable name in model chat test #3837

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
0f42957
Fix variable name in model chat test
EricMichaelSmith Jul 22, 2021
94ef8e4
Try bumping tqdm
EricMichaelSmith Jul 22, 2021
aa9d9d8
Revert
EricMichaelSmith Jul 22, 2021
a900f80
Merge branch 'master' into model-chat-test-name-fix
EricMichaelSmith Jul 23, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 3 additions & 3 deletions tests/crowdsourcing/tasks/model_chat/test_model_chat.py
View file
Open in desktop
Expand Up @@ -107,7 +107,7 @@ def test_base_task(self):
f.write(model_opt_contents)

# Set up the config and database
num_blender_convos = 10
num_convos = 10
args = ModelChatBlueprintArgs()
overrides = [
f'+mephisto.blueprint.{key}={val}'
Expand All @@ -122,12 +122,12 @@ def test_base_task(self):
]
] + [
'mephisto.blueprint.annotations_config_path=${task_dir}/task_config/annotations_config.json',
f'mephisto.blueprint.conversations_needed_string=\"fixed_response:{num_blender_convos:d}\"',
f'mephisto.blueprint.conversations_needed_string=\"fixed_response:{num_convos:d}\"',
f'mephisto.blueprint.chat_data_folder={chat_data_folder}',
'+mephisto.blueprint.left_pane_text_path=${task_dir}/task_config/left_pane_text.html',
'+mephisto.blueprint.max_concurrent_responses=1',
f'mephisto.blueprint.model_opt_path={model_opt_path}',
f'+mephisto.blueprint.num_conversations={num_blender_convos:d}',
f'+mephisto.blueprint.num_conversations={num_convos:d}',
'+mephisto.blueprint.onboard_task_data_path=${task_dir}/task_config/onboard_task_data.json',
'+mephisto.blueprint.task_description_file=${task_dir}/task_config/task_description.html',
]
Expand Down