Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Colored prefixes on chatbot streaming #13

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lionelchg
Copy link
Contributor

A few tweaks on the chatbot_with_streaming.py script:

  • Colored user and assistant prefixes
  • Black formatting on script
  • Add temperature parameter

An example can be found in the screenshot below:
Screenshot 2023-12-14 at 12 56 47

Colored user and assistant prefixes

Black formatting on script

Add temperature parameter
@lionelchg lionelchg changed the title Few improvements on chatbot_with_streaming example Colored prefixes on chatbot streaming Dec 14, 2023
@Bam4d
Copy link
Collaborator

Bam4d commented Dec 18, 2023

#11 Has been merged, so feel free to rebase!

@lionelchg
Copy link
Contributor Author

Hey just solve the conflicts with main branch. I have added colors whenever the model variable pops so that a typical chat now looks like this:

Screenshot 2023-12-19 at 12 24 54

@Bam4d
Copy link
Collaborator

Bam4d commented Jan 15, 2024

Hi @lionelchg, did you still want to contribute these changes? They have failing linting and are a bit out of date.

Thanks

@lionelchg
Copy link
Contributor Author

Hey, will update those later today, completely forgot :)!

@lionelchg
Copy link
Contributor Author

I have merged the updated main and the linting is now OK!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants