-
Notifications
You must be signed in to change notification settings - Fork 306
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
Unable to generate Synth_Pref dataset #506
Comments
Hi @ranarag ! Thanks for opening up the issue. I've already included the missing functions in this PR (review pending but will merge right away): #511 As for the API Status Error: I noticed that you're using AzureAPI. I wonder if you made any changes in the from openai import AzureOpenAI
client = AzureOpenAI(
api_key=api_key,
api_version="2024-08-01-preview",
azure_endpoint=<YOUR ENDPOINT>. # maybe: "https://eteopenai.azure-api.net"
) Internally, the URL we had when we tried Azure is just I'll put an Azure-compatible PR as well but you can try out those changes and let me know! |
To add: So the |
Thanks for the quick resolution! The issue was on my end (related to the API endpoint), which was causing preference annotation to fail. It's fixed now, and the entire pipeline is running smoothly. |
Thank you! Will update the README accordingly :) |
Closing this now 👍 |
Hi,
Thanks for sharing the code for synthectic preference dataset generation!
I have been trying to generate synthetic data using synth_pref, but I encountered some challenges while annotating preferences using scripts.synth_pref.annotate_preferences.
I have been able to convert the annotation mix to the format required by the Batch API by using:
This creates instances like:
However, when I try to annotate the preferences using:
I get the following error:
Am I missing something or doing something incorrectly?
Also, I noticed that the
scripts/synth_pref/parse_preferences.py
appears to be incomplete, as the definitions for the functions the definitions ofbinarize_pref
,compute_mean_rating
andget_rating
are missing.Looking forward to your help regarding the above issues.
The text was updated successfully, but these errors were encountered: