-
Notifications
You must be signed in to change notification settings - Fork 571
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
Save bools role playing #35
Conversation
@@ -27,8 +27,6 @@ def main() -> None: | |||
print( | |||
Fore.CYAN + | |||
f"Specified task prompt:\n{role_play_session.specified_task_prompt}\n") | |||
print(Fore.MAGENTA + |
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.
In RolePlaying ctor specify else branches for if with_task_specify:
to set self.specified_task_prompt = None
.
For if with_task_planner:
- the same.
@@ -43,7 +41,7 @@ def main() -> None: | |||
print_text_animated(Fore.GREEN + | |||
f"AI Assistant:\n\n{assistant_msg.content}\n\n") | |||
|
|||
if "<CAMEL_TASK_DONE>" in user_msg.content: |
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.
What about making "CAMEL_TASK_DONE" a class constant like self.task_done_keyword. You have it in other places
if "<CAMEL_TASK_DONE>" in user_msg.content:
Also what's the point of removing <> ?
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.
message_dict['termination_reason'] = "<CAMEL_TASK_DONE>"
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.
Sometimes the user replies with CAMEL_TASK_DONE without <>. So I remove it. Making it a class constant sounds good to me. I will fix it in following PRs.
All good, you can merge |
No description provided.