[Bug]:top_p is assigned to top_k

[adzhua]

Is there an existing issue for the same bug?

• I have checked the existing issues.

Branch name

latest

Commit ID

c3b232

Other environment information

No response

Actual behavior

Why is top_p assigned to top_k on line 330 of rag/llm/chat_model.py?

Expected behavior

No response

Steps to reproduce

Why is top_p assigned to top_k on line 330 of rag/llm/chat_model.py?

Additional information

No response

[KevinHuSh]

Ollama seems only accept topk.

[adzhua]

Ollama seems only accept topk.

It can accept the top_p parameter; I’ve tested it.
reference link: https://github.com/ollama/ollama/blob/ce1fb4447efc9958dcf279f7eb2ae6941bec1220/examples/python-json-datagenerator/readme.md?plain=1#L45