[GH-ISSUE #10771] Possible bug at llama/llama.go #7076

Closed
opened 2026-04-12 19:00:24 -05:00 by GiteaMirror · 1 comment
Owner

Originally created by @DarkCaster on GitHub (May 19, 2025).
Original GitHub issue: https://github.com/ollama/ollama/issues/10771

What is the issue?

Sorry for the possibly stupid question, I was studying the code and came across this (see image).

Why is the PenaltyPresent parameter from SamplingParams not used by C.struct_common_sampler_cparam, but instead PenaltyFreq is used twice in a row? Looks like a typo for me.

Image

Relevant log output


OS

No response

GPU

No response

CPU

No response

Ollama version

No response

Originally created by @DarkCaster on GitHub (May 19, 2025). Original GitHub issue: https://github.com/ollama/ollama/issues/10771 ### What is the issue? Sorry for the possibly stupid question, I was studying the code and came across this (see image). Why is the `PenaltyPresent` parameter from `SamplingParams` not used by `C.struct_common_sampler_cparam`, but instead `PenaltyFreq` is used twice in a row? Looks like a typo for me. ![Image](https://github.com/user-attachments/assets/4910e9c1-972d-4ec7-b866-d6bb4f21216d) ### Relevant log output ```shell ``` ### OS _No response_ ### GPU _No response_ ### CPU _No response_ ### Ollama version _No response_
GiteaMirror added the bug label 2026-04-12 19:00:24 -05:00
Author
Owner

@woojh3690 commented on GitHub (May 20, 2025):

Maybe related this issue.

https://github.com/ollama/ollama/issues/10767#issue-3072240517

<!-- gh-comment-id:2893578336 --> @woojh3690 commented on GitHub (May 20, 2025): Maybe related this issue. https://github.com/ollama/ollama/issues/10767#issue-3072240517
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#7076