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

add llm_max_tokens to init_content.py #571

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zjjzyl
Copy link

@zjjzyl zjjzyl commented Jul 15, 2024

Description

To ensure that the settings.yaml configuration is respected, I recommend adding the llm_max_tokens parameter to the init_content.py file. This would allow the settings.yaml to include the llm_max_tokens parameter, enabling customization of the local search as intended.

Related Issues

#570

Checklist

  • I have tested these changes locally.
  • I have reviewed the code changes.
  • I have updated the documentation (if necessary).
  • I have added appropriate unit tests (if applicable).

@zjjzyl zjjzyl requested a review from a team as a code owner July 15, 2024 15:03
@AlonsoGuevara AlonsoGuevara requested a review from a team as a code owner July 24, 2024 21:53
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