Default to a sparse 64 x 48 world at 5% occupancy - #22
Merged
Merged
Conversation
Pierre's previews and population studies have used 64 x 48 at 5% occupancy since #8/#9; make that the launch default (153 agents, groups 39/38/38/38) instead of 32 x 24 at 30%. Tests that set their own dimensions but inherited the old 30% occupancy now pin it explicitly, so their populations, fixtures and assertions stay unchanged. The launch test now checks the default dimensions, occupancy and initial group counts. 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.
Change
Pierre asked to make the sparse world used in previews and studies since #8/#9 the launch default:
Only
ExperimentConfig::default()changes. Preset mix, maintenance, seed, 500-tick duration, pacing and all model rules are unchanged.--width,--heightand--occupancystill override the default.Tests
Several Rust and browser tests set their own dimensions but inherited the former 30% occupancy. With the new default, an 8 × 6 world would drop from 14 to 2 agents, which would silently weaken them. These tests now pin
occupancy: 300_000/--occupancy 0.3, so their populations, thefsm-seed1-tick10fixture and every assertion stay as they were. Tests that use the complete default configuration now exercise the new default.ordinary_launch_uses_the_mixed_fsm_populationnow also checks the default dimensions, occupancy, 153 initial agents and 39/38/38/38 groups. It fails onmainwithleft: (32, 24, 300000) right: (64, 48, 50000).Docs
README and
docs/development.mddescribe the new default. The two running examples drop their now-redundant--width 64 --height 48 --occupancy 0.05flags. Thepopulation_tracestudy command keeps explicit dimensions for reproducibility. The unreferenced files indocs/screenshots/still show the older world and were not regenerated.Verification (local Windows, Rust 1.95.0)
cargo fmt --check, Clippy in both feature modes with-D warnings: passedcargo testwithout default features, withweb, and in release: 77 / 95 / 77 passednpm run check: 7 passedgit diff --check: cleanheadless --ticks 0reportswidth=64 height=48 occupancy=0.050000 initial_count=153cargo audit,npm audit. These are left to the Linux and Windows CI jobs for this commit.🤖 Generated with Claude Code