• Stefy Lanza (nextime / spora )'s avatar
    write the learned verdict to disk, not just to memory · 5fbc19ae
    Stefy Lanza (nextime / spora ) authored
    record_resident_fits called save_models(), which writes the config manager's
    own view of models.json -- a view that never saw this change. Verified live
    2026-10-29: after a run that learned the model does not fit, the flag was set
    in memory (the next load in the same process correctly skipped the resident
    attempt, 110.6s instead of 865.3s) while models.json still read
    
        measured_resident_fits = None
    
    so every restart would have paid the ~16 minute discovery again.
    
    persist_model_field is the right call and is what record_vram_delta already
    uses for measured_vram_gb: it re-reads models.json from disk, sets one field
    and writes back, so a stale in-memory copy cannot revert a UI edit made
    meanwhile. It also targets the entry by config_id, so two configs sharing one
    checkpoint path do not overwrite each other's measurement.
    Co-Authored-By: 's avatarClaude Opus 5 <noreply@anthropic.com>
    Claude-Session: https://claude.ai/code/session_014T3rZozi1Ehk96NG1WesgC
    5fbc19ae
manager.py 316 KB