-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature/writing process revision solve the retrieval of writing process #179
Conversation
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.
I also do believe there exists some merging errors in that
@@ -21,7 +22,6 @@ | |||
read_paper_prompting, | |||
review_paper_prompting, | |||
review_score_prompting, | |||
summarize_ideas_prompting, |
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.
for example, this line of code should not be deleted. It is included in the updated main branch
submissions[agent.profile.name] = paper | ||
self.db.update(cls=ResearchPaperSubmission, conditions={}, updates=submissions) | ||
|
||
submissions: Dict[str, PaperProfile] = {} |
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.
Check the main branch, the submission should be in ResearchPaperSubmission type instead of PaperProfile type.
It makes sense for the rest of the logic change.
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 you create a new PR related to changes in assign_roles and environment logic change using match? Thanks
Sure! Sorry for the delay in response. I will reorganize the |
already supported in agent-base |
Closes #
π Description
β Checks
type/descript
(e.g.feature/add-llm-agents
)βΉ Additional Information