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

Lightrag Server Launch fix. #842

Merged
merged 6 commits into from
Feb 18, 2025
Merged

Lightrag Server Launch fix. #842

merged 6 commits into from
Feb 18, 2025

Conversation

YanSte
Copy link
Contributor

@YanSte YanSte commented Feb 18, 2025

This pull request addresses and hide unnecessary temporary lib using PM usage.

It also includes testing upon launching lightrag-server to verify stability.”

  • Removing tqdm library
    • Why is mostly using in some Notebook, in production code we don't need.

@YanSte YanSte marked this pull request as draft February 18, 2025 16:14
@YanSte YanSte requested a review from LarFii February 18, 2025 19:47
@YanSte YanSte marked this pull request as ready for review February 18, 2025 21:14
model,
prompt,
system_prompt=None,
history_messages=[],
**kwargs,
) -> Union[str, AsyncIterator[str]]:
stream = True if kwargs.get("stream") else False
reasoning_tag = kwargs.pop("reasoning_tag", None)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

never use, in all project.

@YanSte YanSte merged commit 2c99dfc into HKUDS:main Feb 18, 2025
1 check passed
@YanSte YanSte deleted the server branch February 18, 2025 21:16
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.

1 participant