-
Notifications
You must be signed in to change notification settings - Fork 50
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
Swap Flask API for FastAPI #78
Comments
Also FastAPI have an awesome automatic documentation generation : |
Thanks for linking that @OlivierCavadenti! Any chance you want to implement this upgrade for us? We would really value your contribution! |
I can check want I can do ! |
Hi @dgarnitz , I will love take this take on |
Go for it! |
Let me know if I can help! |
Hi @dgarnitz, I was trying to maintain the abstraction of the request to get API Keys and files. I am having issues with handling the files. This is sample representation of the file Also with this approach we will lose better auto docs generation support |
@datgerhardt Can you provide me with more context around the bug you ran into? Perhaps some screenshots or a code snippet |
Any update? |
Thanks for your patience, I had the Internet outage in Ghana due to a problem with undersea cable for past few weeks |
Right now we are running Flask with Gunicorn using PM2. There are a few problems with this:
We should swap Flask for FastAPI and (if possible in one PR) swap PM2 for something more appropriate.
Note: We currently depend on Flask-sql alchemy for our DB ORM. We will need to swap this when migrating away from Flask. This may also require changes to the DB services that are also dependencies for the worker and vdb upload worker
The text was updated successfully, but these errors were encountered: