search

valentinfrlch / ha-llmvision

Let Home Assistant see!
Apache License 2.0
186 stars 8 forks source link

Custom OpenAI integration: logs #83

Open smkrv opened 1 week ago

smkrv commented 1 week ago

I found another error with the custom OpenAI integration: if there are multiple, the logs always write the data of the first one created.

And also: if you specify: https://%full_path_url%, the /v1 is still added, so it is correct to specify only the host with the protocol and port. This may be a feature, but I thought I would mention it.

valentinfrlch commented 5 days ago

As suggested in #64, I am going to implement the python openai library instead of using http requests directly. This should also fix this issue. It might take a while though, as I am rewriting the entire request_handlers atm.