renkelvin

Results 31 comments of renkelvin

Hey @ashpreetbedi , any updates with this issue?

Seems it's only used here: https://github.com/phidatahq/phidata/blob/1302da0de264e3419a2c40883ed8e53de8ed794b/phi/assistant/assistant.py#L1227

Hi @ashpreetbedi , any updates to this issue? Or anything I can help with?

Sure, do you mean test OpenAILike with GPT-4?

`OpenAIChat` works fine with GPT-4. ``` llm=OpenAIChat( model="gpt-4", max_tokens=ai_settings.default_max_tokens, temperature=ai_settings.default_temperature, ), ``` `OpenAILike` failed with 404, which is expected since there is no "gpt-4" model on the BASE_URL. ``` llm=OpenAILike(...

> Seems it's only used here: > > https://github.com/phidatahq/phidata/blob/1302da0de264e3419a2c40883ed8e53de8ed794b/phi/assistant/assistant.py#L1227 btw, seems `num_chats` is used here.

Hi @ashpreetbedi , I found out the issue should on the API side. The platform we use doesn't support `["number", "null"]` as the function call parameter type. `num_chats` has a...

Thanks @ashpreetbedi , set `use_tools=False` solves the issue for me. I noticed that `read_chat_history` is overridden here(https://github.com/phidatahq/phidata/blob/0278000a9196e7e6771da22144aa9a2f3b5036ea/phi/assistant/assistant.py#L279) if `use_tools` is set to `True`. I wonder if it's intended?

Thanks for reporting this. Please plus one if you need this feature, which will help us prioritizing.

The 5 minutes is enforced by `expirationTime`, while the 30s is to refresh the token if it will expire in 30s.