Skip to content

Conversation

@mikegros
Copy link
Collaborator

@mikegros mikegros commented Jan 7, 2026

This will be a draft but making a subclass of the execution agent that is specialized to run simulations. Need to test it on a wider base of things, but based on some scripting that worked well and tested with the DCOPF problem from the simulation use workflow.

Why this is a draft:

  • Documentation steps still dont work quite right, I want to refine them and it may result in major changes.
  • Likely requires a separate PR (I will tag when I've pushed) to be merged for this to work properly.

I want it up here for others to test though.

…, build run documents, run simulations, and analyze output.
…equire merging in a PR that I will push up soon to fix some aspects of the execution agent.
@mikegros mikegros marked this pull request as draft January 7, 2026 23:00
@mikegros
Copy link
Collaborator Author

mikegros commented Jan 8, 2026

I this this might require #150 and so that should probably be resolved first. I developed #150 to address wider problems that I came across working on this PR

@mikegros
Copy link
Collaborator Author

I think I want to do 2 things before I take this out of draft status:

  • Expand the graph to have a dedicated node or tool for documentation and node for simulation execution
  • Add a simulation agent test to the tests folder

PR 150 caused a problem with MCP tool use because of the sync invoke inside the tool_use node
Needed to revert to the ToolNode, but that required some better structure in the fix
from PR 150.

This appears to work fully, but need to run the tests and get others to take a shot at it.
…ing. One node for documentation, one node for simulation setup, execution, and analysis. This also has a small change to the write code and edit code tools to ensure they can write markdown files with code snippets.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants