Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add bias-shades #37

Merged
merged 18 commits into from
Nov 9, 2022
Merged

Conversation

oskarvanderwal
Copy link

@oskarvanderwal oskarvanderwal commented May 2, 2022

This task implements the bias-shades dataset for French and English. I'll add the other languages, when the respective prompts are available.

@oskarvanderwal oskarvanderwal reopened this Nov 2, 2022
@oskarvanderwal oskarvanderwal marked this pull request as ready for review November 2, 2022 15:14
Removed language subsets that have no prompts yet.
@oskarvanderwal
Copy link
Author

This PR depends on the French and English prompts to be added in promptsource.

Copy link
Collaborator

@jon-tow jon-tow left a comment

Choose a reason for hiding this comment

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

@oskarvanderwal Thanks for putting this together! I've added bias-shades to the task registry and did a little refactoring to match similar tasks.

I ran into one minor issue - for some reason, the Spanish convert_to_stereotype template does not appear to be working properly

    text, _ = self.prompt_template.apply(doc)
ValueError: not enough values to unpack (expected 2, got 1)

These errors almost always stem from the promptsource side. If you can confirm this, then we can go ahead and merge this.

@oskarvanderwal
Copy link
Author

@jon-tow, thanks for your work! I have asked the author of the Spanish prompts to review the problematic template. I'm pretty sure the problem is coming from the promptsource side, so those prompts should be checked.

@jon-tow jon-tow merged commit 8cea2f4 into bigscience-workshop:master Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants