-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat: sequence task workforce #2055
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
base: master
Are you sure you want to change the base?
Conversation
| "technology used, innovation, and potential impact." | ||
| ), | ||
| ), | ||
| Task( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we add some comments on these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
like step by step run all task ....
|
thanks @nitpicker55555 ,left some comment |
Wendong-Fan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @nitpicker55555 for the contribution, and apologies for the delayed review comment.
The current implementation appears to process a sequence of top-level tasks within the workforce, rather than allowing for a pre-defined, fixed sequence of subtasks for a single overarching goal. As it stands, each task in the sequence is still subject to the workforce's internal decomposition process.
The user's requirement, as I understand it, is to support a scenario where subtasks are explicitly defined and executed in a specific order without further decomposition by the workforce. This would give more direct control over the execution flow.
Maybe allowing the Task to specify which worker node should handle it could offer more granular control. Happy to discuss this further
14bff5d to
1243997
Compare
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
1243997 to
9c61604
Compare
9c61604 to
53f48a2
Compare
1243997 to
7b06d3b
Compare
|
@Wendong-Fan I've updated the Workforce so it now processes pre-defined subtasks directly instead of always decomposing them, and I also added a way for user to assign specific subtasks to the exact worker user wants. And I've put multi_hop_example.py to show how it all works, looking forward to your feedback |
thanks @nitpicker55555 ! Will check the update later, please remember to update the PR label |
Wendong-Fan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested with the example code seems it's hanging, could @nitpicker55555 help checking the reason?
|
@Wendong-Fan CI passed now, this failure is not related to current pr and happened many times because of the test_alpaca_collector file, |
hey @nitpicker55555 , I mean it was hanging when I ran |
|
@nitpicker55555, any updates? |

Description
Support for processing a sequence of tasks in the
Workforce, where each task can share context (e.g., results, memory) with others.process_task_sequenceandprocess_task_sequence_asyncmethods toWorkforceshared_contextandshared_channelto persist information across tasksshared_contextwith the patterntask_{task.id}process_taskThis addresses collaborative task execution as proposed in Issue #1832.
Checklist
Fixes #1832(required)pyproject.tomlanduv lock