Skip to content

Add non-wrapping grid option to vivarium#17

Closed
Copilot wants to merge 6 commits intodevelopfrom
copilot/add-non-wrapping-grid-option-again
Closed

Add non-wrapping grid option to vivarium#17
Copilot wants to merge 6 commits intodevelopfrom
copilot/add-non-wrapping-grid-option-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 26, 2026

  • Fix GPU zero-padding bug in setup.ts: testNeighbor, testIdInPack, comparePointWithId, comparePointWithPoint, checkPointCount must use std.select to return element ID 0 (not ids[0]) when OOB in non-wrapping mode
  • Fix JSDoc in vivarium.ts: "Defaults to true" → "Defaults to false"
  • Fix docs vivarium.mdx: remove "fewer neighbors" language, describe zero-padding correctly
  • Fix test "count at corner sees only in-bounds neighbors": fix comment (8 positions always, OOB are zero-padded to element 0)
  • Redesign test "count at edge does not include out-of-bounds as neighbors": grid all-a with count(a, 8) is a bad proof because OOB also map to a; replace with a test that demonstrates OOB are element 0 not b
  • Fix test "full board dies (overcrowding)": add { wrapping: true } — with correct zero-padding, 3x3 corner cells only see 3 alive in-bounds neighbours and survive
  • Run tests

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: OrangeNote <5088762+OrangeNote@users.noreply.github.com>
Copilot AI changed the title [WIP] Add option for non-wrapping grid in vivarium instantiation Add non-wrapping grid option to vivarium Feb 26, 2026
Co-authored-by: OrangeNote <5088762+OrangeNote@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants