-
Notifications
You must be signed in to change notification settings - Fork 1.2k
docs: Add guide to migrating from Agents to Responses #4375
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
base: main
Are you sure you want to change the base?
Conversation
…e OpenAI-compatible Responses API
raghotham
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to run the mcp server as part of the notebook as well?
|
@raghotham , I tried this out in my local copy and it seems to work. It seemed like a good idea to me at first, but in trying it out, I am finding that it adds a lot of complexity. Here are two issues that I am running into:
If these are not disqualifying, I think there are two different approaches we could take each with their own pros and cons: Option 1: Continue to point to the AI Alliance repo for where to get the NPS server. Tell users to download the server from there and install the dependencies themselves (in the environment where the notebook is running, since it kicks off the server). The main drawback of this approach is that we haven't gotten much closer to making the notebook "just work" out of the box -- you still need to do most of the same setup, we just save users the one command to start the server at the end. Option 2: Make a fork of the NPS server that lives in this directory and then have the notebook install all its dependencies for you (via a requirements.txt file or similar). The main drawbacks are that it adds clutter here (the server code, its requirements.txt) and also that now there are two separate forks of this server that are likely to get out of sync with each other. What do you think? Should we go ahead anyway and if so which option do you prefer? |
|
@raghotham , I wanted to follow up again on this since it is still open. Do you have a view on which path seems better to you? I guess I lean toward just leaving it as is because of the added complexity of managing an MCP server in a notebook, but I don't feel very strongly. |
What does this PR do?
Sample notebook shows how to migrate from the old Agents API to the newer Responses API.
Closes #3542
Test Plan
I ran the notebook. The outputs are included in the notebook itself.