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

Assistants_API_overview_python notebook updated for the latest openai package #1464

Merged
merged 5 commits into from
Jan 7, 2025

Conversation

the-praxs
Copy link
Contributor

@the-praxs the-praxs commented Oct 12, 2024

Summary

Closes #1463

Fixes the Assistants_API_overview_python notebook by updating the code for the latest openai package. This also fixes the code logic where multiple ids for the tools were present but not submitted to the thread

Motivation

Necessary for the community who refer to this code to understand the Assistants API.

@the-praxs the-praxs changed the title fix for latest openai version with updated code Assistants_API_overview_python notebook updated for the latest openai package Oct 12, 2024
@the-praxs
Copy link
Contributor Author

the-praxs commented Oct 12, 2024

Requesting review @logankilpatrick @shyamal-anadkat @colin-openai

@the-praxs
Copy link
Contributor Author

Requesting review - @justonf @lspacagna-oai

Copy link
Contributor

@lspacagna-oai lspacagna-oai left a comment

Choose a reason for hiding this comment

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

Checked updates work correctly locally. Thanks for contributing!

@lspacagna-oai lspacagna-oai merged commit 686e0d7 into openai:main Jan 7, 2025
@the-praxs the-praxs deleted the fix-assistants branch January 8, 2025 10:16
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.

[PROBLEM] Assisstants API Overview notebook broken due to outdated code
2 participants