Chat interface updates - new flags for multi-line input, model choice selection, updated paragraph wrapping for output and option flag to disable, and minor bugfixes #230
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Ah, also added a mini timeout feature, for when requests occasionally go bad. Probably should change the commit message to reflect that. Will wait for feedback |
PeteHemery
force-pushed
the
chat-interface-updates
branch
from
ceb526f
to
f7a68ec
Compare
|
split commits up into manageable chunks. hope some are good for cherry picking |
PeteHemery
force-pushed
the
chat-interface-updates
branch
from
f7a68ec
to
451a3c0
Compare
PeteHemery
changed the title
PeteHemery
changed the title
PeteHemery
force-pushed
the
chat-interface-updates
branch
5 times, most recently
from
c61e9e3
to
2e12df4
Compare
|
always with the "just one more thing" with git ;) FYI, my usage style with these changes is: I've also been making some improvements to the save chat functionality too. being able to name saved chats as you start them, and allowing branches with reworking from a certain point. need to save the chat_vdqs along the way. open for a discussion, if it'd be of any interest? |
PeteHemery
force-pushed
the
chat-interface-updates
branch
from
2e12df4
to
6a515ba
Compare
added 4 commits
June 18, 2024 00:42
Previous output:
```python import click
def wrap_text_with_newlines(text, width=78): paragraphs = text.split('\n')
wrapped_paragraphs = [click.wrap_text(paragraph, width=width) for paragraph in
paragraphs] return '\n'.join(wrapped_paragraphs)
text = "This is a sample text.\n\nIt has multiple paragraphs.\nEach paragraph
should be wrapped separately." wrapped_text = wrap_text_with_newlines(text,
width=78) print(wrapped_text) ```
Updated output:
```python
import click
def wrap_text_with_newlines(text, width=78):
paragraphs = text.split('\n')
wrapped_paragraphs = [click.wrap_text(paragraph, width=width) for
paragraph in paragraphs]
wrapped_text = '\n'.join(wrapped_paragraphs)
return wrapped_text
text = "This is a sample text.\nIt has multiple lines.\nEach line should be
wrapped separately."
wrapped_text = wrap_text_with_newlines(text, width=20)
print(wrapped_text)
```
Allow the user to enter multiple lines in a single prompt. Remind the user that the EOF command key will finish the input.
When dealing with large amount of text as a prompt, it's easier to edit command line history when copying and pasting a number of paragraphs. When asking coding questions it's useful to be able to edit multi-line input after pasting, which can't be done in the interactive console. This new functionality will assume multi-line input is enabled, and check between interactive console and piping from stdin behaviour. Since longer prompts take longer to respond, increase the timeout to 30s
PeteHemery
force-pushed
the
chat-interface-updates
branch
from
6a515ba
to
e96beb4
Compare
|
The code is overcomplicated. |
Repository owner
deleted a comment from
PeteHemery
Jun 19, 2024
Repository owner
locked as spam and limited conversation to collaborators
Jun 19, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Following on the inspiration to share back to the community from PR 227, I offer these improvements.
The commit messages should have most of the info there.
I did modify/comment out this line
text = resp_answer #click.wrap_text(resp_answer, width=78, preserve_paragraphs=True)
since the formatted text returned does look right when it prints mardown code.
Let me know what you think.