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

feat(platform): Support opening graphs with version and execution id #9332

Conversation

kcze
Copy link
Contributor

@kcze kcze commented Jan 25, 2025

Currently it's only possible to open latest graph from monitor and see the node execution results only when manually running. This PR adds ability to open running and finished graphs in builder.

Changes 🏗️

Builder now handles graph version and execution ID in addition to graph ID when opening a graph. When an execution ID is provided, node execution results are fetched and subscribed to in real time. This makes it possible to open a graph that is already executing and see both existing node execution data and real-time updates (if it's still running).

  • Use graph version and execution id on the builder page and in useAgentGraph
  • Use graph version on the execute_graph endpoint
  • Use graph version on the websockets to distinguish between versions
  • Move formatEdgeID to utils; it's used in useAgentGraph.ts and in Flow.tsx

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Opening finished execution restores node results
    • Opening running execution restores results and continues to run properly
    • Results are separate for each graph across multiple tabs

For configuration changes:

  • .env.example is updated or already compatible with my changes
  • docker-compose.yml is updated or already compatible with my changes
  • I have included a list of my configuration changes in the PR description (under Changes)
Examples of configuration changes
  • Changing ports
  • Adding new services that need to communicate with each other
  • Secrets or environment variable changes
  • New or infrastructure changes such as databases

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end size/l labels Jan 25, 2025
Copy link

netlify bot commented Jan 25, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 6b1f35a
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67a4e54fba0b9c00081885b2

Copy link

netlify bot commented Jan 25, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 6b1f35a
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67a4e54f06bb9e000893636f

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 26, 2025
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 28, 2025
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

Copy link

deepsource-io bot commented Jan 31, 2025

Here's the code health analysis summary for commits 797916c..6b1f35a. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ Success
❗ 60 occurences introduced
View Check ↗
DeepSource Python LogoPython✅ Success
❗ 1 occurence introduced
🎯 1 occurence resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@kcze kcze marked this pull request as ready for review January 31, 2025 14:00
@kcze kcze requested a review from a team as a code owner January 31, 2025 14:00
@kcze kcze requested review from aarushik93 and majdyz and removed request for a team January 31, 2025 14:00
Copy link
Contributor

@majdyz majdyz left a comment

Choose a reason for hiding this comment

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

This is exciting! sorry for missing this PR review

autogpt_platform/backend/backend/cli.py Show resolved Hide resolved
autogpt_platform/backend/backend/server/conn_manager.py Outdated Show resolved Hide resolved
autogpt_platform/frontend/src/hooks/useAgentGraph.ts Outdated Show resolved Hide resolved
autogpt_platform/frontend/src/hooks/useAgentGraph.ts Outdated Show resolved Hide resolved
autogpt_platform/frontend/src/hooks/useAgentGraph.ts Outdated Show resolved Hide resolved
autogpt_platform/frontend/src/hooks/useAgentGraph.ts Outdated Show resolved Hide resolved
@aarushik93
Copy link
Contributor

removing myself, Zamil's reviewed

@aarushik93 aarushik93 removed their request for review February 6, 2025 15:24
Copy link
Contributor

github-actions bot commented Feb 6, 2025

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Feb 6, 2025
@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Feb 6, 2025
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@github-actions github-actions bot added size/xl and removed size/l labels Feb 6, 2025
@kcze kcze requested a review from majdyz February 6, 2025 16:38
@kcze kcze enabled auto-merge February 7, 2025 10:16
@kcze kcze added this pull request to the merge queue Feb 7, 2025
Merged via the queue into dev with commit 1a1fe7c Feb 7, 2025
27 checks passed
@kcze kcze deleted the kpczerwinski/open-2023-support-for-opening-graph-with-version-execution-id branch February 7, 2025 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/backend AutoGPT Platform - Back end platform/frontend AutoGPT Platform - Front end size/xl
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants