Add previous_execution_id field to DatasetRow/scenario#3457
Add previous_execution_id field to DatasetRow/scenario#3457devin-ai-integration[bot] wants to merge 4 commits intomainfrom
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-Authored-By: vincent@vellum.ai <0426vincent@gmail.com>
Co-Authored-By: vincent@vellum.ai <0426vincent@gmail.com>
Co-Authored-By: vincent@vellum.ai <0426vincent@gmail.com>
Co-Authored-By: vincent@vellum.ai <0426vincent@gmail.com>
f9ed1ed to
4e706e6
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
|
@TirmanSidhu brought up an interesting point last week where we may be able to infer the |
Adds
previous_execution_idfield toDatasetRow(scenario) in the SDK, enabling users to reference a previous Workflow Execution for initial State loading. This field is now supported in both the Python SDK and the TypeScript codegen for round-trip serialization.Updates since last revision:
previous_execution_idthroughWorkflowSandboxRunner.run()toworkflow.stream(), so running a sandbox with a dataset row that includesprevious_execution_idwill now correctly load prior stateWorkflowSandboxRunnerpassesprevious_execution_idto the workflow streamReview & Testing Checklist for Human
previous_execution_idactually loads state from the previous execution (the unit test only verifies the parameter is passed, not the end-to-end behavior)previous_execution_idparameterNotes