Repository navigation
The container builds its own models; the Settings build button follows the host plan - #207
Merged
Merged
Conversation
…s the host plan
start.bat checks and builds the models before the desktop server starts.
The container starts the server directly, so a fresh NAS install came up
with no bakes until someone opened a shell.
- services/model_autobuild.ensure_main_models: with AUTO_BUILD_MODELS (set
in the Dockerfile), the lifespan starts it in the background on a
configured install. It builds what the main host is missing from
host_plan("main") while the server already answers; a first build is
20+ GB, past any health check.
- It logs what it builds and what failed (ERROR), and names a split PC's
missing bakes without pulling them onto it.
- It reports the embedding identity (ERROR on a weights mismatch).
- It never raises.
- setup_wizard.build_ollama_models(embed_model=): callers on the event
loop pass the model, so no database read happens there.
- routers/setup build_models_endpoint follows host_plan("main"): no
pitcher on a split install's NAS, and the host's own window.
- docs/NAS_MOVE.md step 6: start the container and watch the log, no
manual build.
New tests/test_model_autobuild.py (13 checks). Battery 228/228.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
NAS_MOVE.md step 6 now quotes the container log line ('embedding weights
match the stored vectors').
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why:
start.batchecks and builds the models before the desktop server starts. The container starts the server directly, so a fresh NAS install came up with no bakes until someone opened a shell. Don, rightly: that's nonsense for a container.services/model_autobuild.ensure_main_modelsAUTO_BUILD_MODELS(set in the Dockerfile), the lifespan starts it in the background on a configured install.host_plan("main"). The server answers right away; a first build is 20+ GB, far past any health check.Also
setup_wizard.build_ollama_models(embed_model=): callers on the event loop pass the model, so no database read happens on the loop./api/setup/build-models) followshost_plan("main"): no 17.7 GB pitcher on a split install's NAS, and the host's own window.docs/NAS_MOVE.mdstep 6: start the container and watch the log, instead of a manual build.Tests: new
tests/test_model_autobuild.py(13 checks):test_agent_onlyfollows the reworded runbook. The full battery is green (228/228).🤖 Generated with Claude Code