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

Model chat: space out annotation buckets #4275

Merged
merged 5 commits into from Jan 4, 2022

Conversation

EricMichaelSmith
Copy link
Contributor

@EricMichaelSmith EricMichaelSmith commented Dec 23, 2021

Patch description

  • For the model_chat crowdsourcing task, if the text of the annotation categories is very long, you can set the "show_line_breaks" field (see task_config/annotations_config.json) to true in order to put a line break between the checkbox and text string of each category. This was useful recently when we had a test with very long questions for each checkbox.

Testing steps
Test command:

python parlai/crowdsourcing/tasks/model_chat/run.py \
mephisto.architect.port=3005 \
mephisto.task.maximum_units_per_worker=1000000 \
mephisto.task.task_name=${TASK_NAME}

"show_line_breaks": false:
Screen Shot 2021-12-23 at 4 54 37 PM

"show_line_breaks": true:
Screen Shot 2021-12-23 at 4 55 40 PM

@EricMichaelSmith EricMichaelSmith marked this pull request as ready for review December 23, 2021 22:49
Copy link
Contributor

@meganung meganung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah this is a great option that improves the UX nicely!

@EricMichaelSmith EricMichaelSmith merged commit 22fd071 into main Jan 4, 2022
@EricMichaelSmith EricMichaelSmith deleted the add-breaks-between-buckets branch January 4, 2022 14:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants