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

Expose ImportWorkflow Endpoint #3581

Open
wants to merge 7 commits into
base: 0.2
Choose a base branch
from

Conversation

ofimbres
Copy link

@ofimbres ofimbres commented Sep 29, 2024

Why are these changes needed?

As of Sep 29th, there is an existing endpoint in autogen studio backend for exporting an entire workflow (including agents, models and skills), but no import workflow. This PR exposes a new endpoint for importing a workflow based on the output generated by export workflow and persist it into the database.

Checks

@jackgerrits jackgerrits added the proj-studio Related to AutoGen Studio. label Sep 30, 2024
@ekzhu ekzhu changed the base branch from main to 0.2 October 2, 2024 18:24
@jackgerrits jackgerrits added the 0.2 Issues which are related to the pre 0.4 codebase label Oct 4, 2024
@rysweet rysweet added the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 10, 2024
@victordibia
Copy link
Collaborator

victordibia commented Oct 18, 2024

Hi,

@ofimbres , Thanks for this PR.
Adding new datamodels tends to break existing db schema.
There is some work to be done to enable automatic database upgrade e..g, detect schema differences and update the database. (not specific to this PR, but useful overall for ags)
Any chance you have explored this or have some thoughts?

@ofimbres
Copy link
Author

ofimbres commented Oct 29, 2024

Hi,

@ofimbres , Thanks for this PR. Adding new datamodels tends to break existing db schema. There is some work to be done to enable automatic database upgrade e..g, detect schema differences and update the database. (not specific to this PR, but useful overall for ags) Any chance you have explored this or have some thoughts?

@victordibia This schema uses the same schema used for export. Export uses a new schema that is not defined by pydantic, as a dict. Now that there is a new version, I will look into the new architecture changes and rebase them

@github-actions github-actions bot removed the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.2 Issues which are related to the pre 0.4 codebase proj-studio Related to AutoGen Studio.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants