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

Enhanced customization of API parameters for better control over the behavior of the Ollama model. #1659

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

Conversation

moyueheng
Copy link

Features

  • Enhanced customization of API parameters for better control over the behavior of the Ollama model.
  • Added repeat_penalty parameter to the _llama_api_kwargs to reduce the likelihood of repetitive responses.

Feature Docs

Influence

  • This change allows developers to fine-tune the model output based on specific needs by adjusting temperature and repeat_penalty dynamically through config.
  • It should not affect existing implementations as long as the config object is properly updated to include repetition_penalty.

Result

  • Unit tests were updated to cover new configuration options.
  • All tests pass with the new changes, ensuring that existing functionalities remain unaffected.

Other

  • Ensure that the config object in your application is updated to include repetition_penalty to avoid runtime errors.
  • The implementation follows the recommended practices as per the latest Ollama API guidelines.

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