Fix setuptools build failure; add /auto-research ratchet loop#2
Open
tetiioo wants to merge 2 commits into
Open
Fix setuptools build failure; add /auto-research ratchet loop#2tetiioo wants to merge 2 commits into
tetiioo wants to merge 2 commits into
Conversation
uv run/uv sync failed because setuptools saw both data/ and rest_sim/ as top-level packages with no discovery config to disambiguate. Scope package discovery to rest_sim explicitly. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Adapts the Karpathy-style AutoResearch ratchet pattern to this repo: the only mutable artifact is .claude/agents/restaurant-manager.md (analogous to train.py), the frozen harness is rest_sim/* plus /play-month, and the ratchet metric is end-of-month net profit gated by hard constraints (mean_reputation >= 4.0, mean_satisfaction >= 0.75) instead of the engine's own smooth-penalty score_eur. Results are logged to an untracked, append-only results.tsv. Co-Authored-By: Claude Sonnet 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
1 participant
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.
Summary
uv run/uv syncbuild failure: setuptools saw bothdata/andrest_sim/as top-level packages under flat-layout discovery and refused to build. Added[tool.setuptools.packages.find] include = ["rest_sim*"]topyproject.tomland committed the resultinguv.lock..claude/commands/auto-research.md, a Karpathy-style AutoResearch ratchet loop adapted to this repo: the sole mutable artifact is.claude/agents/restaurant-manager.md(analogous totrain.pyin the original ML version), the frozen harness isrest_sim/*+/play-month, and the ratchet metric is end-of-month net profit (total_net_profit_eur) gated by hard constraints —mean_reputation >= 4.0andmean_satisfaction >= 0.75— rather than the engine's own smooth-penaltyscore_eur. Each iteration edits the prompt, plays a fixed-seed month via therestaurant-managersubagent, pulls a freshscorecard, and keeps the commit only if profit improves without violating either constraint.results.tsv(append-only ratchet log) is now gitignored, matching the pattern used for the untrackedgame/*runtime files.Test plan
uv run python -m rest_sim --helpsucceeds (previously failed with "Multiple top-level packages discovered in a flat-layout")uv run --with pytest pytest -q— 47/47 tests pass/auto-researchrun on a clean checkout (not yet run end-to-end — each iteration is a full ~30-day subagent playthrough, so this is a heavier loop than the ML version; worth a maintainer sanity pass before relying on it)