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

Re-added the beta assistants API #470

Merged
merged 8 commits into from
Apr 15, 2024
Merged

Conversation

CongquanHu
Copy link
Contributor

Based on the latest changes to the library, I added APIs related to assistants to make PR merging easier.
1.Added beta assistants API (mostly)
2.The APIs used in the assistants demo flow were tested

@CongquanHu
Copy link
Contributor Author

CongquanHu commented Jan 2, 2024

@kayhantolga
I looked at the latest library changes, indeed the previous PR and dev content is diff too much, very easy to cause conflict, so I decided to readjust.

I fork again and delete the previous fork warehouse (I don't know if there is any other way, I haven't thought of any other way for the time being, please let me know if there is, thank you).
Assistants API for openai beta was added again, conflicting files were checked, and response adjustments were made. Fortunately, there were not many conflicting files, and I quickly adjusted them.

Most of the API has written test code, and some parts I will add later after self-testing.
Hopefully, this new PR will facilitate branch merging.

Looking forward to your reply, thank you

@CongquanHu
Copy link
Contributor Author

I'm done. Please let me know if there is anything wrong.

@dmki
Copy link

dmki commented Jan 7, 2024

Typo: Insturctions
Also, can you replace Chinese with English, please?

@CongquanHu
Copy link
Contributor Author

CongquanHu commented Jan 8, 2024

Typo: Insturctions Also, can you replace Chinese with English, please?

@dmki Thank you for reminding me.
I have changed the Chinese content of Instructions into English in the last commit.
image
image

@giovanni-a
Copy link

Thanks for this PR! I tested it, it works great! Small typo here: "StaticValues.AssistatntsStatics" instead of "StaticValues.AssistantsStatics"

@CongquanHu
Copy link
Contributor Author

Thanks for this PR! I tested it, it works great! Small typo here: "StaticValues.AssistatntsStatics" instead of "StaticValues.AssistantsStatics"

Thank you for reminding me. It's fixed.

@JKetelaar
Copy link

Any update on this?

@kayhantolga kayhantolga added this to the 8.0.2 milestone Apr 10, 2024
@kayhantolga kayhantolga merged commit bfcaf80 into betalgo:dev Apr 15, 2024
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.

5 participants