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.

Sign up for GitHub

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

Jump to bottom

In-app Chatbot demo #68

Merged
merged 11 commits into from
Mar 19, 2024
Merged

In-app Chatbot demo #68

merged 11 commits into from
Mar 19, 2024

Conversation

elijahbenizzy
Copy link
Contributor

No description provided.

@elijahbenizzy
Copy link
Contributor Author

elijahbenizzy commented Mar 12, 2024 •
edited
Loading

Rough sketch.
image

Required for production:

  • Get the image generation to save to an s3 bucket behind a CDN for demos (should be able to reuse image-telephone? Create a new one?). Or just save locally... And use in static assets. Probably cleaner/easier.
  • Add interactivity
    • Test out interactive endpoint
    • Add the ability to press Send
    • Rename Send to have the classic paper airplane emoji that everyone likes
    • Get it to show the full chat history on click + prompt user clicked in
    • Add ability to create new one
      • Enusre it doesn't break
  • Fix hacks
    • Ensure we can import without openAI/create a client
    • Fix classpath so it just works when running
    • Add it so users know what to do if they have to create a client
      • This should be a fixed message and a specific step -- maybe add Error?
  • Fix commit history up
  • Add dependencies to pyproject.toml

@elijahbenizzy elijahbenizzy force-pushed the examples branch 4 times, most recently from 7e2245c to 840e7b9 Compare March 15, 2024 21:44
@elijahbenizzy elijahbenizzy changed the title WIP for in-app demos In-app Chatbot demo Mar 15, 2024
@elijahbenizzy
Copy link
Contributor Author

Bypassed this one: #69

skrawcz
skrawcz approved these changes Mar 16, 2024
View reviewed changes
Copy link
Contributor

@skrawcz skrawcz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems fine

@elijahbenizzy
Copy link
Contributor Author

elijahbenizzy commented Mar 16, 2024 •
edited
Loading

Blocked on #73 -- this leverages that so I'm going to use it. Will also need to rebase/reimplement some small parts.

@elijahbenizzy
Updates .gitignore to include more common files
678467b
@elijahbenizzy
Cleans up unecessary type variable
2b12a02
@elijahbenizzy
Adds temporary trick to see if something exists -- this is getting re…
c973ff9 
…placed by persistence in #73
@elijahbenizzy
Adds step to check for openai_api_key in environment
c10b7be 
This way people get a clear error message
@elijahbenizzy
Updates server to have chatbot example
db49900 
We still need to fix this up, it doesn't work with the python path, and
examples are not in the server.
@elijahbenizzy
Adds chatbot demo to Burr
a0919a6 
This relies on the new endpoint. Counter is still a WIP but that should
be easy.
@elijahbenizzy
Fixes up chatbot server to work on install
170d359 
We use a symlink to install the `examples` file under `burr/examples`.
This alllows us to keep it separate but have it be a package under
`burr`.
@elijahbenizzy
Adds contribution instructions
af96424
@elijahbenizzy
Updates chatbot demo to work with new API key checking function
1a47805
@elijahbenizzy
Adds builder accessor in application
9d9051c 
This way you can pull it out and keep building.
@elijahbenizzy
Updates chatbot in-app demo to use new persistence capability
6602c86 
This also chagnes the tracker to enable that.
@elijahbenizzy elijahbenizzy merged commit 2eca07d into main Mar 19, 2024
6 checks passed
@elijahbenizzy elijahbenizzy deleted the examples branch March 19, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skrawcz skrawcz skrawcz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@elijahbenizzy @skrawcz