Issue #315: Display full completion in CLI #1068
Open
+16
−4
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR addresses issue #315. The idea is introducing a new
--verbose
flag to the CLI to display the entire completion response.Implementation
--verbose
to CLICLIChatCompletionCreateArgs
verbose_output
flag to static method_create
Caveats
Since
verbose
is not achat.completions.create
argument, we can't pass it toCompletionCreateParams
. In my opinion it seems ok to have a flag for now, but I'm sure there's a cleaner way of doing it in the future if new args are introduced that aren't parameters for completions endpoint.Notes
Do we want this for
_stream_create
?Thanks in advance 😀 @RobertCraigie