Skip to content

refactor(llm): extract topk_per_keyword & topk_return_results to .env#154

Merged
imbajin merged 9 commits intoapache:mainfrom
MrJs133:extract_key
Jan 9, 2025
Merged

refactor(llm): extract topk_per_keyword & topk_return_results to .env#154
imbajin merged 9 commits intoapache:mainfrom
MrJs133:extract_key

Conversation

@MrJs133
Copy link
Contributor

@MrJs133 MrJs133 commented Jan 6, 2025

  1. Extract argument topk_per_keyword to file .env
  2. Extract argument topk_return_results to file .env
  3. Rename max_items to max_graph_items (argument of function query_graphdb)

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jan 6, 2025
@github-actions github-actions bot added the llm label Jan 6, 2025
@dosubot dosubot bot added the enhancement New feature or request label Jan 6, 2025
@MrJs133 MrJs133 changed the title refactor(llm): Extract argument topk_per_keyword of SemanticIdQuery a to file .env refactor(llm): Extract argument topk_per_keyword of SemanticIdQuery to file .env Jan 7, 2025
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jan 7, 2025
@MrJs133 MrJs133 changed the title refactor(llm): Extract argument topk_per_keyword of SemanticIdQuery to file .env refactor(llm): Extract argument topk_per_keyword and topk_return_results to file .env Jan 7, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 9, 2025
@imbajin imbajin merged commit 9239799 into apache:main Jan 9, 2025
11 checks passed
@imbajin imbajin changed the title refactor(llm): Extract argument topk_per_keyword and topk_return_results to file .env refactor(llm): extract topk_per_keyword & topk_return_results to .env Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer llm size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants