Skip to content

[Major] Remove dependency on OpenAI#6

Merged
ashwinzyx merged 2 commits intomainfrom
model-flexibility-for-eval
Aug 15, 2024
Merged

[Major] Remove dependency on OpenAI#6
ashwinzyx merged 2 commits intomainfrom
model-flexibility-for-eval

Conversation

@aravind10x
Copy link
Contributor

  • Added selector options for
    • Eval framework
    • Eval Embedding
    • Eval LLM
    • Non-OpenAI selections for synthetic data generator
  • Fixed bug of getting 403 while checking URL validity
  • Fixed bug of Azure openai (model provider prefix mismatch)
  • As testing, removed openai-api-key, and ran end-to-end with Azure.

- Added selector options for
  - Eval framework
  - Eval Embedding
  - Eval LLM
  - Non-OpenAI selections for synthetic data generator
- Fixed bug of getting 403 while checking URL validity
- Fixed bug of Azure openai (model provider prefix mismatch)
@aravind10x aravind10x requested a review from ashwinzyx August 14, 2024 11:32
@aravind10x aravind10x changed the title [Major] Break dependency on OpenAI [Major] Remove dependency on OpenAI Aug 14, 2024
- Change model param type from str to object in `generate_data`
- Removed double temperature keyword in groq part in `llmConfig`
- Fixed Azure prefix to AzureOAI
Copy link
Contributor

@ashwinzyx ashwinzyx left a comment

Choose a reason for hiding this comment

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

LGTM

@ashwinzyx ashwinzyx merged commit da98be2 into main Aug 15, 2024
@aravind10x aravind10x deleted the model-flexibility-for-eval branch August 15, 2024 11:48
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.

2 participants