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

Pull out settings for model chat onboarding #4274

Merged
merged 2 commits into from
Jan 4, 2022

Conversation

EricMichaelSmith
Copy link
Contributor

Patch description

  • Allow for setting "min_correct", "max_incorrect", and "max_failures_allowed" fields in the JSON file passed to mephisto.blueprint.onboard_task_data_path in order to specify how many onboarding questions workers can pass/fail on while still passing onboarding, as well as how many times they are allowed to re-take the onboarding before being soft-blocked
    • See task_config/onboard_task_data.json for an example of how this looks
    • Enabled by the recent refactor of the model chat annotation bucket schema by @meganung !
  • Removes duplicated checking of onboarding annotation answers on the backend

Testing steps
Ran test command:

python parlai/crowdsourcing/tasks/model_chat/run.py \
mephisto.architect.port=3003 \
mephisto.task.maximum_units_per_worker=1000000 \
mephisto.blueprint.onboarding_qualification=gihuecnirnfbijdddtgunjhiuifbrvbn

UI when setting "max_failures_allowed": 10 and then getting some responses wrong:
Screen Shot 2021-12-23 at 3 34 39 PM

@EricMichaelSmith EricMichaelSmith marked this pull request as ready for review December 23, 2021 20:39
@EricMichaelSmith EricMichaelSmith changed the title [WIP] Pull out settings for model chat onboarding Pull out settings for model chat onboarding Dec 23, 2021
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.

LGTM!

@@ -22,7 +22,9 @@ The following flags can be passed in to specify filepaths for overriding the tex

## Onboarding

In `worlds.py`, modify `ModelChatOnboardWorld.check_onboarding_answers()` to change the worker selection criteria.
Set the `"min_correct"`, `"max_incorrect"`, and `"max_failures_allowed"` fields in the JSON file passed to `mephisto.blueprint.onboard_task_data_path` in order to specify how many onboarding questions workers can pass/fail on while still passing onboarding, as well as how many times they are allowed to re-take the onboarding before being soft-blocked. (See `task_config/onboard_task_data.json` for an example.)
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@EricMichaelSmith EricMichaelSmith merged commit e686a02 into main Jan 4, 2022
@EricMichaelSmith EricMichaelSmith deleted the clean-up-onboarding branch January 4, 2022 14:45
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.

3 participants