search

aandrew-me / tgpt

AI Chatbots in terminal without needing API keys
GNU General Public License v3.0
2.04k stars 171 forks source link

Add help text for --url #252

Closed johnd0e closed 7 months ago

johnd0e commented 7 months ago

Missed in #249

johnd0e commented 7 months ago

BTW, current implementation demand to specify full endpoint url - https://api.openai.com/v1/chat/completions. This is not typical and not convenient. It would make more sense sticking to more common way, like: https://api.openai.com/v1.

And it also would make sense to use environment variable OPENAI_BASE_URL, which is also pretty common.

aandrew-me commented 7 months ago

Its api endpoint only for open ai, it would be misleading if that's not mentioned

johnd0e commented 7 months ago

Fixed

aandrew-me commented 7 months ago

BTW, current implementation demand to specify full endpoint url - https://api.openai.com/v1/chat/completions. This is not typical and not convenient. It would make more sense sticking to more common way, like: https://api.openai.com/v1.

And it also would make sense to use environment variable OPENAI_BASE_URL, which is also pretty common.

Regarding the variable, I agree. Regarding the URL, I will check