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

fix(llm): update config in time after apply changes in UI #90

Merged

Conversation

returnToInnocence
Copy link
Contributor

@returnToInnocence returnToInnocence commented Sep 30, 2024

Brief description

  1. Fixed an issue where rag web demo could not update system configuration in time after configuration file changes in file hugegraph-llm/src/hugegraph_llm/demo/rag_demo/app.py
    • graph config, the picture is as follows
      image
    • prompt, the picture is as follows
      image
    • rag with hugegraph, as follows
      image

To avoid errors that result in an empty return value in file hugegraph-llm/src/hugegraph_llm/demo/rag_demo/configs_block.py

  1. Adjust the order of functions and classes in file hugegraph-llm/src/hugegraph_llm/config/config.py

  2. Adjust the format of the prompt to make it easier to read in file hugegraph-llm/src/hugegraph_llm/config/config_data.py

  3. Add the ability to save prompt to the configuration file after a button click to the kg build section in file hugegraph-llm/src/hugegraph_llm/demo/rag_demo/vector_graph_block.py, the position as follows
    image

issue

fix some of the #91

@github-actions github-actions bot added the llm label Sep 30, 2024
@returnToInnocence returnToInnocence marked this pull request as ready for review October 9, 2024 11:20
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working labels Oct 9, 2024
@imbajin imbajin changed the title fix(rag): Fixed an issue where rag web demo could not update system configuration in time after configuration file changes fix(llm): update config in time after apply changes in UI Oct 10, 2024
@imbajin imbajin merged commit 8bb13f9 into apache:main Oct 15, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working llm size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants