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] Seperate out create and run node methods #52

Merged
merged 9 commits into from
Dec 12, 2024

Conversation

adi-kmt
Copy link
Collaborator

@adi-kmt adi-kmt commented Dec 4, 2024

This solves the issue

@adi-kmt adi-kmt marked this pull request as ready for review December 7, 2024 13:06
@adi-kmt adi-kmt force-pushed the feat/seperate_out_create_and_run_node_methods branch from 6e385dd to 713561b Compare December 9, 2024 09:41
# Create command
create_parser = subparsers.add_parser("create", help="Execute create command.")
create_parser.add_argument("agent", help="Select the agent to run")
create_parser.add_argument("-e", "--environment_nodes", help="Environment nodes to store data during agent runs.")
Copy link
Contributor

Choose a reason for hiding this comment

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

Should create also take worker_nodes as an arg?

create_parser = subparsers.add_parser("create", help="Execute create command.")
create_parser.add_argument("agent", help="Select the agent to run")
create_parser.add_argument("-e", "--environment_nodes", help="Environment nodes to store data during agent runs.")

Copy link
Contributor

Choose a reason for hiding this comment

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

Should we remove worker_nodes and environment_nodes from the run_parser args?

@richardblythman richardblythman force-pushed the feat/seperate_out_create_and_run_node_methods branch from b29d819 to e3886f7 Compare December 12, 2024 00:01
@richardblythman richardblythman merged commit f50ca52 into main Dec 12, 2024
@richardblythman richardblythman deleted the feat/seperate_out_create_and_run_node_methods branch December 12, 2024 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Implement new create() methods for the Agent(), Environment(), Orchestrator(), Tool() classes
3 participants