Skip to content

UPSTREAM PR #18374: llama_fit_params: return enum for fail vs. error#705

Open
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18374-branch_JohannesGaessler-llama-fp-status-code
Open

UPSTREAM PR #18374: llama_fit_params: return enum for fail vs. error#705
loci-dev wants to merge 1 commit intomainfrom
upstream-PR18374-branch_JohannesGaessler-llama-fp-status-code

Conversation

@loci-dev
Copy link
Copy Markdown

Mirrored from ggml-org/llama.cpp#18374

See discussion in ggml-org/llama.cpp#18236 .

This PR makes it so that llama_params_fit returns an enum rather than a boolean in order to differentiate between success, failure of the algorithm, and hard errors due to e.g. the model file not being found.

@loci-dev loci-dev force-pushed the main branch 4 times, most recently from 3091d19 to f7cdf84 Compare December 26, 2025 18:11
@loci-dev loci-dev force-pushed the upstream-PR18374-branch_JohannesGaessler-llama-fp-status-code branch from 732351d to 9bd5942 Compare December 26, 2025 20:36
@loci-review
Copy link
Copy Markdown

loci-review bot commented Dec 26, 2025

Explore the complete analysis inside the Version Insights

I apologize for the error. It seems there's an issue with the summary agent configuration. The system is unable to generate the summary report for the specified project details:

  • Project ID: 2621b8c0-b5ce-11f0-b333-453f42058aa1
  • Report ID: ff6162fb-e897-4913-8588-f92caa008e6c
  • Version ID: 86a36c63-dff8-45f3-8106-6b8ffdc9a4a2
  • Base Version ID: 1b37d0a3-2d73-4ca4-bc16-73c34a215de1
  • Repository: llama.cpp
  • Owner: auroralabs-loci
  • Pull Request: 705

The error indicates that the required agent node is not properly configured in the system. You may need to:

  1. Check if the summary reporting service is running
  2. Verify the agent configuration
  3. Contact system administrators to resolve the "project_summary_agent" node issue

Is there anything else I can help you with, or would you like me to try a different approach?

@loci-dev loci-dev force-pushed the main branch 21 times, most recently from 3dbad97 to 220f305 Compare December 29, 2025 17:08
@loci-dev loci-dev force-pushed the main branch 30 times, most recently from cf2ca8e to 9297073 Compare January 5, 2026 08:13
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