Skip to content
This repository was archived by the owner on Dec 8, 2023. It is now read-only.

feat: add grit migration script for openai#14

Merged
morgante merged 1 commit intogetgrit:mainfrom
SarthakSingh31:openai
Sep 15, 2023
Merged

feat: add grit migration script for openai#14
morgante merged 1 commit intogetgrit:mainfrom
SarthakSingh31:openai

Conversation

@SarthakSingh31
Copy link
Contributor

No description provided.

chat_completion = await aclient.chat.completions.create(model="gpt-3.5-turbo", messages=[{"role": "user", "content": "Hello world"}])
```

# Change openai import to Both
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's also add a case verifying that multiple calls to each API (ex. two Completions.create) reuse the same client (ie. we don't create a new client for each instance).

@morgante morgante changed the title Added grit migration script for openai feat: add grit migration script for openai Sep 15, 2023
Copy link
Contributor

@morgante morgante left a comment

Choose a reason for hiding this comment

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

Merging as is so we can start testing, but there's more to be done.

@morgante morgante merged commit f2c3b79 into getgrit:main Sep 15, 2023
@SarthakSingh31 SarthakSingh31 deleted the openai branch September 15, 2023 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants