-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[DRAFT] Convert autogenstudio
to new serializable config language
#5200
[DRAFT] Convert autogenstudio
to new serializable config language
#5200
Conversation
autogenstudio
to new serializable config language
Hi @EItanya , There are a few thoughts here
These are deep reaching changes and probably should all land in unison. As discussed on the other thread, can you help with shepherding the other MCP Httptools PR, getting it merged ? That would be a huge help if we can already test these things robustly in python code first, write documentation before we hammer out studio plumbing? |
Hey, thanks for the quick response, I think it's worth explaining a bit of my motivation here. I know this will not be the actual final PR for this feature, I'm lacking a ton of context on the project overall, not to mention I am not a very good UI engineer 😆. I am going to be going on vacation starting Tuesday and this functionality is more to unblock some of my colleagues and prove out some things to myself, rather than anything official. I decided to share it in case it is of any use to you in starting out as it does seem to work. I am working on a project which is hoping to leverage the backend for In terms of the specific feedback:
|
Ah .. got it. |
Sounds good to me, no rush! Definitely feel free to use this branch |
This is the branch where I am tracking work on updating the backend. (lots of similarity to what you have here) and UI. UI piece still needs a bit of work. Will update here again when it is ready in some form. |
If you aren't actively using this branch should I close the PR for clarity? |
Yes, we can close for now |
@victordibia and @ekzhu and @jackgerrits as reviewers?
Why are these changes needed?
This is a draft PR to begin the server side work of migrating
autogen-studio
over to the new serializable config style.Related issue number
#5172
Checks