Skip to content

Pr 5175 nowarmup verboseprompt #5733

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 32 commits into
base: micn/embedded-llama-server
Choose a base branch
from
Open

Pr 5175 nowarmup verboseprompt #5733

wants to merge 32 commits into from

Conversation

@ATrueLight4
Copy link
Contributor

@ATrueLight4 ATrueLight4 commented Nov 14, 2025

Summary

--no-warmup loads larger models faster, --verbose-prompt, see the actual prompts

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Manual

Related Issues

Relates to #5175
Discussion: LINK (if any)

Screenshots/Demos (for UX changes)

Before:

After:

Submitting a Recipe?

Email:

jamadeo and others added 30 commits November 11, 2025 08:20
@jamadeo
use npx in just generate-openapi (block#5664)
90449e7
@jamadeo
the right script (block#5671)
9345319
@tiensi
Swapped out to_string_lossy with display for user facing text (block#…
23f26fb 
…5666)

Signed-off-by: Vincent Huang <[email protected]>
@jamadeo
Update release instructions (block#5662)
80da813
@alexhancock
chore: upgrade rmcp to 0.8.5 (block#5673)
a09f2e4
@alexhancock
governance: update MAINTAINERS.md to reflect new maintainers (block#5675
f0ecf56 
)
@jamadeo
bump openapi version directly (block#5674)
5110d32
@angiejones
teach copilot our CI (block#5672)
fb68966
@alexhancock
feat: SessionManager integration for acp sessions (block#5657)
a07023c
@blackgirlbytes
docs: add GOOSE_PATH_ROOT environment variable documentation (block#5678
92d86c0 
)

Signed-off-by: Rizel Scarlett <[email protected]>
@emma-squared
Document recent goose PRs (block#5683)
a2c862b
@Abhijay007
fix : action icons overlap session title in chat history (block#5684)
775bd90 
Signed-off-by: Abhijay007 <[email protected]>
@github-actions
chore(release): release version 1.14.0 (minor) (block#5676)
133e357 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@maniksurtani
chore: Update governance (block#5660)
56feb15
@angiejones
New maintainers (block#5685)
6f0fcd2
@dianed-square
docs: session storage migration (block#5682)
5648799
@alexhancock
fix: do not take into account gitignore in developer mcp (block#5688)
7e3ad08
@alexhancock
fix: make image processing work in github copilot provider (block#5687)
cd33b23
@blackgirlbytes
Unlist VS Code extension tutorials from MCP and experimental sections (…
d00b7fa 
…block#5677)
@Rodriguespn
feat: add Supabase MCP server to registry (block#5629)
4a6e682 
Signed-off-by: Pedro Rodrigues <[email protected]>
@DOsinga
Ollama improvements (block#5609)
0df4ad3 
Co-authored-by: Douwe Osinga <[email protected]>
@taniandjerry
chore: Update governance to include Discord (block#5690)
c842621 
Co-authored-by: Tania Chakraborty <[email protected]>
@katzdave
Fix: Always show autocompact threshold ui (block#5701)
a747ba7 
Co-authored-by: Copilot <[email protected]>
@alexhancock
fix: improve and simplify tool call chain rendering (block#5704)
c2b8c64
@tiensi
Added transaction commits to multi sql functions in session_manager (b…
e1b5af5 
…lock#5693)

Signed-off-by: Vincent Huang <[email protected]>
@blackgirlbytes
Fix documentation-only change detection for push events (block#5712)
8fe7be3
@blackgirlbytes
hackathon banner (block#5710)
7e3a754
@dianed-square @angiejones
docs: mcp sampling support (block#5708)
f233f9f 
Co-authored-by: Angie Jones <[email protected]>
@angiejones
docs: removing double announcements (block#5714)
bdfcbf0
@catbilyeu @claude @katzdave
Fix context progress bar not resetting after /clear command (block#5652)
075cbb2 
Signed-off-by: Cat Bilyeu <[email protected]>
Co-authored-by: Claude <[email protected]>
Co-authored-by: David Katz <[email protected]>
@michaelneale michaelneale changed the base branch from main to micn/embedded-llama-server November 14, 2025 01:46
@michaelneale michaelneale requested a review from a team as a code owner November 14, 2025 01:46
@ATrueLight4
Copy link
Contributor Author

ATrueLight4 commented Nov 15, 2025

so I am in a quandary... I think my PR for adding --verbose-prompt and --no-warmup should be closed and deleted...

I noticed that the llama-embedded branch was updated and stdio and stderr are now going to null, so no need for --verbose or --verbose-prompt... And if I add the --no-warmup to a new PR, for not spawning a window for llama-server when running the GUI in Windows the PR I just created is no longer needed...

Next I just tested not spawning a window, which never happened in CLI, and the change now no longer spawns a window for the llama-server in GUI under Windows and with the --no-warmup both CLI and GUI starts the larger gpt-oss in time to respond to first prompt given...

Problem... GUI version of goose still doesn't kill the llama-server process at least under Windows. The CLI did, does, and still does, stop the llama-server when the goose CLI stops

But with no window means user would need to open taskmanager or something to kill the llama-server process

I can create a PR with this change to not spawn a window for the llama-server under Windows, but making sure the GUI version of goose kills the llama-server process should probably occur first...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.