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.

Sign up for GitHub

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

Jump to bottom

improve some RAG function UT(tests) #167

Open
imbajin opened this issue Feb 20, 2025 · 6 comments · May be fixed by #192
Open

improve some RAG function UT(tests) #167

imbajin opened this issue Feb 20, 2025 · 6 comments · May be fixed by #192
Assignees
@yanchaomei
Labels
enhancement New feature or request

Comments

@imbajin
Copy link
Member

imbajin commented Feb 20, 2025 •
edited
Loading

Refer https://github.com/apache/incubator-hugegraph-ai/tree/main/hugegraph-llm/src/tests

We do lack UT for the LLM module, the reliability of this code cannot be guaranteed, and a basic testing framework is urgently needed to improve it
@dosubot dosubot bot added the enhancement New feature or request label Feb 20, 2025
@yanchaomei
Copy link

Hello, I'm new to the community and interested in handling this task. Would that be possible?

@imbajin
Copy link
Member Author

imbajin commented Feb 25, 2025

Hello, I'm new to the community and interested in handling this task. Would that be possible?

@yanchaomei Sure, assigned to u~

You can also use LLM as much as possible to assist in generating a large number of functions.

We can divide the testing into several parts/steps. The core goal of the first phase is to prioritize testing to ensure that the functionality is normal and there will be no serious/obvious errors caused by modifying the code

@chiruu12
Copy link

chiruu12 commented Mar 1, 2025

Hello sir if this issue had not been resolved yet can I start working on it too?
The last PR I was working on just has some pytest issues left I will work on this pr and the one I was assigned earlier at the same tome?

@imbajin
Copy link
Member Author

imbajin commented Mar 3, 2025

Hello sir if this issue had not been resolved yet can I start working on it too? The last PR I was working on just has some pytest issues left I will work on this pr and the one I was assigned earlier at the same tome?

@chiruu12 Sure, before the previous claiming dev @yanchaomei submitted PR, you can participate in some of its tasks, and then you can also work with this classmate in the future

@chiruu12
Copy link

chiruu12 commented Mar 3, 2025

@imbajin Ok sir I will complete the pr I have at hand and then move forward to working on this one

yanchaomei added a commit to yanchaomei/incubator-hugegraph-ai that referenced this issue Mar 5, 2025
@yanchaomei
feat(llm):improve some RAG function UT(tests)
ba85fbc 
fix apache#167
@yanchaomei yanchaomei linked a pull request Mar 5, 2025 that will close this issue
Open
@yanchaomei
Copy link

Hi, I have submitted the related PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yanchaomei yanchaomei

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(llm):improve some RAG function UT(tests) yanchaomei/incubator-hugegraph-ai
3 participants
@imbajin @yanchaomei @chiruu12