Skip to content

Update llama-cpp-rs to 0.1.155 - #35

Merged
pierotofy merged 2 commits into
LibreTranslate:mainfrom
PolynomialDivision:llama-cpp-rs-1-55
Sep 1, 2026
Merged

pierotofy merged 2 commits into
LibreTranslate:mainfrom
PolynomialDivision:llama-cpp-rs-1-55

Conversation

@PolynomialDivision

Copy link
Copy Markdown
Contributor

While updating, also upgrade all the other dependencies.

@pierotofy

Copy link
Copy Markdown
Member
error[E0061]: this function takes 5 arguments but 4 arguments were supplied
   --> ltengine/src/llm.rs:247:13
    |
247 |             LlamaSampler::penalties(64, 1.0, 0.0, 0.0),
    |             ^^^^^^^^^^^^^^^^^^^^^^^     --- argument #2 of type `i32` is missing

While updating, also upgrade all the other dependencies.
Update CUDA/Ubuntu/Rust, remove the NCCL workaround, and shrink the runtime image.
@PolynomialDivision
PolynomialDivision marked this pull request as ready for review September 1, 2026 08:51
@PolynomialDivision

Copy link
Copy Markdown
Contributor Author

@pierotofy Yes, sorry! I opened this PR late at night while the Docker build was still running, then fell asleep. When I woke up, I saw the error message. I should have put it into draft mode. xD Now everything works and I updated also the docker.

@pierotofy
pierotofy merged commit 9b748dd into LibreTranslate:main Sep 1, 2026
1 check passed
@pierotofy

Copy link
Copy Markdown
Member

No worries :), thanks for the PR!

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