Repository navigation
Faster screens: keep the opened sheet, cache Review's list, instant button feedback - #53
Merged
Merged
Conversation
…utton feedback - sheet.py: the opened spreadsheet and its tabs are kept 5 minutes (one request per read instead of three); opened again on a new login, an expiring token, a refused request, a renamed or deleted tab - Apply: answered from the rows read last; opening it re-reads both tabs in the background; sheet_version moves only when a read finds a change; each tab draws as soon as its answer comes - Review: the list and the taste model are kept until data/, settings or the sheet rows change, and made ahead at start and after decisions - Buttons show busy at once and ignore double clicks; row-menu sheet changes show a working toast; switching screens shows Loading at once Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…no second reload that closes an open menu) 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.
Apply and Review were slow to open: every load re-opened the Google Sheet (three requests) and Review retrained the taste model on every load and search.
sheet.py): the opened spreadsheet and its tabs are kept 5 min (sheet.OPEN), so a read is one request (~0.2 s instead of ~0.8 s). Rows are always read fresh. Opened again on a new login, an expiring access token, a refused request, or a renamed/deleted tab.sheet_versionmoves only when a read finds a change, and each tab draws as soon as its own answer comes. Apply now opens in ~0.02 s.data/, settings or the sheet rows change (gui.review_lists). The taste model retrains only when ratings or job text change (jobs.taste_for). Lists are made ahead at start and after decisions. Review now answers in ~0.006 s (was 1.2–5 s).Docs: GUI.md (new SPEED section), DEPLOY.md, ARCHITECTURE.md, PIPELINE.md, CLAUDE.md.
Tests: new checks in test_sheet (the kept sheet), test_intern (retrain only on change) and test_gui (
sheet_version, Review re-made after a change). The full suite passes locally. In an earlier runtest_demohung under load; it passed in every run after that.test_guifailed once in ~13 runs, cause not found.🤖 Generated with Claude Code