search

kardolus / chatgpt-cli

ChatGPT CLI is a versatile tool for interacting with ChatGPT models via OpenAI and Azure, as well as with models from Perplexity AI. It offers streaming, query mode, and history tracking for seamless, context-aware interactions. With extensive configuration options, it’s designed for both users and developers to create a tailored GPT experience.
MIT License
503 stars 34 forks source link

[Fix] Add missing max_token in request query #34

Closed nopeless closed 6 months ago

nopeless commented 6 months ago

fixes #31

kardolus commented 6 months ago

@nopeless thank you so much for this! I will either merge this in and then fix the tests after or I will suggest a patch. I'm hoping to find some time today or tomorrow.

nopeless commented 6 months ago

whoops, forgot tests existed. I only tested manually

nopeless commented 6 months ago

@kardolus also if it helps with context, i find myself using the query mode for batch jobs. that is why i need the max token on the output

kardolus commented 6 months ago

@nopeless here's a patch that fixes the tests. Do you want to apply it? I can do it to after the merge, but it would be nice to keep the main branch green.

0001-Fix-tests.patch

nopeless commented 6 months ago

@kardolus i can apply that patch but i am also pretty sure that you can modify the pr itself

nopeless commented 6 months ago

allow edits by maintainers is checked

kardolus commented 6 months ago

@nopeless ah gotcha, either is fine by me. I just updated the patch btw :). Added the wrong file earlier.

kardolus commented 6 months ago

Thank you!