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

Add a randomize_conversations to ACUTE blueprint + use #3528

Merged
merged 7 commits into from
Apr 17, 2021
Merged

Conversation

moyapchen
Copy link
Contributor

...cause I want to be able to not randomize conversations.

Context here is that I have chats generated from different models using a fixed set of prompts. I want to be able to compare how different models do on the same prompt and consequently do not want the chats to be randomized.

Test plan:
Print out the pairs value from this function; verify that things align.

...cause I want to be able to not randomize conversations.

Context here is that I have chats generated from different models using a fixed set of prompts. I want to be able to compare how different models do on the same prompt and consequently do not want the chats to be randomized.

Test plan:
Print out the pairs value from this function; verify that things align.
Copy link
Contributor

@EricMichaelSmith EricMichaelSmith left a comment

Choose a reason for hiding this comment

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

Yeah, makes sense! Can you add the default value of this param to the example YAML file? It looks like the crowdsourcing CI checks are failing without it

@moyapchen moyapchen merged commit 5081caa into master Apr 17, 2021
@moyapchen moyapchen deleted the derandomize branch April 17, 2021 16:37
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.

4 participants