-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor code #9
Conversation
You can try out pydantic validator? In sotopia, we may want to use it in place of beartype. |
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 for adding the interface for checking the tool! I'll edit my pull request based on this.
Good point! I will think this further and probably implement this on other pull requests |
Closes #5
📑 Description
In order to keep the agent outputting actions to call the tools, we can basically utilize an internal validator like this:
HAI-Cosys/haicosystem/generation_utils/validation.py
Line 10 in 49ae78f
Using such a mechanism could ensure the structure of the agent output is correct.
To realize this, basically we need to have the tool parser:
HAI-Cosys/haicosystem/tools/tool_parser.py
Line 30 in 49ae78f
This format can do a rule-based check of the output of LLMs.
✅ Checks
type/descript
(e.g.feature/add-llm-agents
)ℹ Additional Information