-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add ReAct agent and a streamlit web app #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Hema,
Great start. 👍🏼
In addition, I would like to suggest:
- Adding dependency files (like requirements.txt or pyproject.toml).
- Including the steps to start both the API service and the Streamlit app.
I was able to start the api service using python react_agent/api.py
. However, I encountered issues starting the Streamlit app.
Thanks for the review Surya! I was going to include a dependency file along with a README describing all the steps in a separate PR. Also, what issues were you having with streamlit? |
Thanks @hemajv . I got this error while starting the webapp:
Error: |
@suppathak ah so to run the streamlit app we just need to execute |
febe581
to
c86044b
Compare
@suppathak @Shreyanand updated the PR with your suggested changes, ptal! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you Hema for the update. Added some comment. ptal!
c86044b
to
25306bb
Compare
@suppathak updated the PR with the fixes/modifications, ptal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @hemajv .👍🏼
In this PR, I've added the code/artifacts for the following:
Related issue - #2
(Note: This is based off our existing internal shadowbot project)