-
Notifications
You must be signed in to change notification settings - Fork 557
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
Docker fix #583
base: main
Are you sure you want to change the base?
Docker fix #583
Conversation
@ParisNeo let's have a discussion about the codebase and the challenges users are facing, how to address them, and how I can help improve the codebase. I need this solution to be in working order for me to be able to accomplish my vision of how I want AI to interact. let's schedule an hour long call to do a deep dive and work through this stuff so we can:
|
Dear @ba2512005, I want to be transparent about both LoLLMS's positioning and my current personal situation:
Innovation History and Impact:
Alternative Recommendations:
Contribution Process:
Personal Situation and Project Constraints:
While I remain committed to LoLLMS, I must prioritize my family's needs in the coming months. This means I'll have significantly less time to dedicate to development and maintenance. I hope you understand that this is a necessary step for maintaining a healthy work-life balance. You are still welcome to contribute to the project following the guidelines above. However, if you're looking to build something for business purposes, I strongly recommend exploring the alternatives mentioned earlier. Thank you for your understanding and continued support of LoLLMS. Despite these temporary constraints, I remain passionate about the project and its innovative contributions to the AI community. Best regards, |
Description
Slight modifications to code to allow for
docker compose up
to work properly.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
Checklist:
Please put an
x
in the boxes that apply. You can also fill these out after creating the PR.Screenshots
If applicable, add screenshots to help explain your changes.
Docker environment and image build works fine,
running the container i get this error:
File "/app/app.py", line 15, in lollms-webui-1 | from fastapi.middleware.cors import CORSMiddleware lollms-webui-1 | ModuleNotFoundError: No module named 'fastapi'
Referring to line 15 of app.py where its trying to use a fastapi import without importing fastapi.
The wierd thing is that I updated the code to remove that yet it is still there..
@ParisNeo can you please test this and let me know if it works on your end or if there's something we have to do to update the lollms_webui package from 5.0.2 to 5.0.3 to have the updated code in the package that is being pulled in the docker compose.