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

Customer chatbot #90

Merged
merged 10 commits into from
Feb 2, 2024
Merged

Customer chatbot #90

merged 10 commits into from
Feb 2, 2024

Conversation

lfunderburk
Copy link
Contributor

@lfunderburk lfunderburk commented Jan 29, 2024

Copy link
Contributor

@edublancas edublancas left a comment

Choose a reason for hiding this comment

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

It's unclear what this example does, please include a README explaining what it does and how to use it.

I tried it asking "what can you do?" and got an error

image

=2.2.0 Outdated Show resolved Hide resolved
examples/docker/fastapi-ai-helper/draft.ipynb Outdated Show resolved Hide resolved
@edublancas
Copy link
Contributor

after trying again, I got this other error:

image

@neelasha23
Copy link
Contributor

It's unclear what type of questions can be asked. For the below queries it responds with another query:

Screenshot 2024-01-30 at 11 30 35 AM Screenshot 2024-01-30 at 11 30 56 AM

Also, can we build a simple UI interface ?

@lfunderburk
Copy link
Contributor Author

after trying again, I got this other error:

image

I tried the same question and it worked ok

Redeploying currently

@lfunderburk
Copy link
Contributor Author

It's unclear what type of questions can be asked. For the below queries it responds with another query:

Screenshot 2024-01-30 at 11 30 35 AM Screenshot 2024-01-30 at 11 30 56 AM
Also, can we build a simple UI interface ?

Added description to app

I have been deploying solara apps the last 2 weeks, I wanted to deploy an API as the API may be used in non-python based websites

@lfunderburk
Copy link
Contributor Author

@edublancas

Redeployed, please give it a few minutes and let me know if the issue persists

examples/docker/fastapi-ai-helper/README.md Show resolved Hide resolved
examples/docker/fastapi-ai-helper/README.md Outdated Show resolved Hide resolved
examples/docker/fastapi-ai-helper/README.md Outdated Show resolved Hide resolved
@lfunderburk
Copy link
Contributor Author

Redeployed

https://calm-violet-6179.ploomberapp.io/docs

If this looks good I will make the changes to the blog

@edublancas edublancas merged commit 2d37445 into main Feb 2, 2024
1 check passed
@edublancas edublancas deleted the customer-chatbot branch February 2, 2024 00:23
@edublancas
Copy link
Contributor

@lfunderburk please keep your commits short. otherwise, if you answer the observations with a commit that has tons of modifications, it takes more time to review them

@lfunderburk
Copy link
Contributor Author

Ok can do

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.

3 participants