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
Addition of types #493
base: master
Are you sure you want to change the base?
Addition of types #493
Conversation
Create an extra type: 'ReasonType' as an optional argument. The 'ReasonType' decides how the crucial elements of the message generated by the system is displayed. The default value is 'Reason', making no change to the output message. Another value, 'REACT', split the output message into reasoning and action, with the former part only involving explanation and the latter part involving function calling.
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
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 as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Hey @huyinan , thanks for the PR! This PR is still in progress and is not ready for review right? If so we can make the status as "draft", when it's ready for review we can reopen it, thanks!
<YOUR_ACTION> should be directly implementing the examples mentioned before. Always end <YOUR_SOLUTION> with: Next request.""") | ||
|
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.
prompt added here seems not clear, should the assistant always add <YOUR_ACTION>
here? In the prompt above action was not mentioned
ACTOR_PROMPT (TextPrompt): A system prompt for the AI actor that takes actions (usually search key words) | ||
based on the outline of several key words from previous output from ASSISTANT. Then search results are |
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.
could you explain what does 'usually search key words' mean here to help me understand? thanks!
camel/agents/react_embodied_agent.py
Outdated
from camel.messages import BaseMessage | ||
from camel.prompts import TextPrompt | ||
from camel.types import ModelType, RoleType, ReasonType | ||
from camel.functions.search_functions import * |
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 would suggest not using * doing import here
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.
OK. This line has been commented out.
role_descriptions_dict (Optional[Dict[str, str]], optional): The | ||
descriptions of the roles. (default: :obj:`None`) | ||
role_descriptions_dict (Optional[Dict[str, str]], optional): A |
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.
repetitive docstring here
@@ -17,7 +17,7 @@ | |||
from camel.agents import ChatAgent | |||
from camel.messages import BaseMessage | |||
from camel.prompts import TextPrompt | |||
from camel.types import ModelType, RoleType | |||
from camel.types import ModelType, RoleType, ReasonType |
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.
why added ReasonType
here but not used?
@@ -23,7 +23,7 @@ | |||
) | |||
from camel.messages import BaseMessage | |||
from camel.responses import ChatAgentResponse | |||
from camel.types import ModelType | |||
from camel.types import ModelType, ReasonType |
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.
why added ReasonType
here but not used?
…cement learning framework
Create an extra type: 'ReasonType' as an optional argument. The 'ReasonType' decides how the crucial elements of the message generated by the system is displayed. The default value is 'Reason', making no change to the output message. Another value, 'REACT', split the output message into reasoning and action, with the former part only involving explanation and the latter part involving function calling.
Description
Motivation and Context
Types of changes
What types of changes does your code introduce? Put an
x
in all the boxes that apply:Implemented Tasks
Checklist
Go over all the following points, and put an
x
in all the boxes that apply.If you are unsure about any of these, don't hesitate to ask. We are here to help!