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

fix: set uvicorn worker to 1 #26

Merged
merged 2 commits into from
Jul 16, 2024
Merged

fix: set uvicorn worker to 1 #26

merged 2 commits into from
Jul 16, 2024

Conversation

ccheng26
Copy link
Contributor

Changes

change amount of worker #'s to 1

Context for reviewers

Chainlit uses Socket.io for backend server connections; multiple workeres are currently unsupported by gunicorn
https://flask-socketio.readthedocs.io/en/latest/deployment.html#gunicorn-web-server

Testing

Run make start
Navigate to localhost:8000/chat and enter a query into the chatbot, response should be returned

@ccheng26 ccheng26 requested review from KevinJBoyer and yoomlam July 16, 2024 18:19
Copy link
Contributor

@yoomlam yoomlam left a comment

Choose a reason for hiding this comment

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

Also revert docker-compose to use uvicorn so that hot-loading is restored

@ccheng26 ccheng26 requested a review from yoomlam July 16, 2024 18:26
@ccheng26 ccheng26 merged commit ed9760b into main Jul 16, 2024
4 checks passed
@ccheng26 ccheng26 deleted the setUvicornWorkerToOne branch July 16, 2024 18:39
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.

2 participants