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

feat: Added webui management, including file upload, text upload, Q&… #647

Merged
merged 5 commits into from
Jan 26, 2025

Conversation

18277486571HYB
Copy link
Contributor

Added webui management, including file upload, text upload, Q&A query, graph database management (can view tags, view knowledge graph based on tags), system status (whether it is good, data storage status, model status, path),request /webui/index.html

… query, graph database management (can view tags, view knowledge graph based on tags), system status (whether it is good, data storage status, model status, path),request /webui/index.html
@LarFii
Copy link
Collaborator

LarFii commented Jan 25, 2025

Thanks for your contributions! It would be helpful to change all the comments to English so that other developers can easily work on the project in the future. Thank you again!

@ParisNeo
Copy link
Contributor

Hi, I can translate them if you want.
And I think we can merge my webui with this one if you don't mind.
I can do that after the pull request is merged.

@18277486571HYB
Copy link
Contributor Author

Whatever,you can do

@18277486571HYB
Copy link
Contributor Author

Do you need any assistance from me?

@ParisNeo
Copy link
Contributor

I just need to wait for it to be accepted before I can fuse it so we can start from a clean slate.

I took a look at your code, I think I can handle it without help :) .
If I really don't understand something I'll just ask you.

Thanks alot.

@LarFii LarFii merged commit 28b139d into HKUDS:main Jan 26, 2025
1 check passed
@ParisNeo
Copy link
Contributor

Well I just checked it. In fact, it is not much more advanced than my webui. The things you add (the graph management) are not implemented so they don't work. Do you plan on adding implementation of that?

Also, you shouldn't use BASE_URL in the lightrag.js script as it is already served from a specific url and you don't need to use any BASI_URLK, just use /the end point and it will work even if you change the server address.

I'm keeping my webui, but I'll add some ideas from yours and i'll also add a placeholder so that you can implement graph visualization if you like that.

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