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

Cleanup of code #885

Merged
merged 1 commit into from
Feb 19, 2025
Merged

Cleanup of code #885

merged 1 commit into from
Feb 19, 2025

Conversation

YanSte
Copy link
Collaborator

@YanSte YanSte commented Feb 19, 2025

Description

  • Remove unsused method
  • Cleaned get_all_labels never implemented everywhere
  • Cleaned import

Changes Made

  • Info print
  • Remove get_all_labels

Checklist

  • Changes tested locally
  • Code reviewed

@YanSte YanSte merged commit 4978379 into HKUDS:main Feb 19, 2025
1 check passed
@@ -1682,11 +1682,6 @@ async def stream_generator():
trace_exception(e)
raise HTTPException(status_code=500, detail=str(e))

# query all graph labels
@app.get("/graph/label/list")
Copy link
Contributor

@ArnoChenFx ArnoChenFx Feb 20, 2025

Choose a reason for hiding this comment

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

This API is also used by the WebUI. We can't remove it.
@YanSte

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Okay.

@ArnoChenFx
Copy link
Contributor

All the APIs used by the WebUI are here: /~https://github.com/HKUDS/LightRAG/blob/main/lightrag_webui/src/api/lightrag.ts

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