feat: Add new llm provider - ppio #1991
Conversation
|
This pr is relatived to add new llm provider: ppio |
|
@saikidev thanks！can you add example and test like 6a39c1a#diff-cccd34af48a94d9606279ca10de626a2a22a552383eacde07d9010fd35585f4e |
|
You should also modify the |
Thanks! And I have fixed. |
Can you help to merge this PR, Thanks a lot! |
yeah,of course，thanks @saikidev |
|
hello @saikidev ，please resolve the conflicts |
fix: parallel tool call sequence bug, upadte doc and readme (camel-ai#2148)
|
@saikidev thanks |
There was a problem hiding this comment.
thanks @saikidev , left some comments below
cc @fengju0213
| def as_dict(self) -> dict[str, Any]: | ||
| config_dict = super().as_dict() | ||
| if "tools" in config_dict: | ||
| del config_dict["tools"] # Currently does not support tool calling | ||
| return config_dict |
There was a problem hiding this comment.
from https://ppinfra.com/docs/model/llm-function-calling I think it's supporting tool call
| self, | ||
| model_type: Union[ModelType, str], | ||
| model_config_dict: Optional[Dict[str, Any]] = None, | ||
| api_key: Optional[str] = None, | ||
| url: Optional[str] = None, | ||
| token_counter: Optional[BaseTokenCounter] = None, |
| @api_keys_required( | ||
| [ | ||
| ("api_key", 'PPIO_API_KEY'), | ||
| ] | ||
| ) |
There was a problem hiding this comment.
we need to add the key to yml config file
|
@Wendong-Fan I noticed that you have already modified my code. Can I ask when it will be released? |
it is already released @saikidev |
But I install CAMEL with all dependencies by running "uv pip install -e .", There is no file about PPIO |
You need to update to the latest version |
5 participants
Description
Add a new llm provider: ppio.
Checklist
Go over all the following points, and put an
xin all the boxes that apply.Fixes #issue-numberin the PR description (required)