Make the pool flow downstream from the fall foot and let the splash foam drift - #29
Merged
Merged
Conversation
…oam drift The owner reported that the pool the waterfall empties into read as stagnant. The pool section of the water shader is now a flow field: a line source along the whole foot of the lower curtain, water leaving it downward and fanning outward, blending into a drain toward the foreground-left. Two cross-faded downstream passes carry the photographed foam and reflections; bubbles at the splash drift and glint gently; open water shows faint streaks. Pool pixels may pull texture from the pool or the curtain, never from stones. Falls, masks, still image, lifecycle and uniforms are unchanged. Speeds are art speeds, not physical water. Owner review during the build replaced a first point source (read as a spring in the middle) with the line source, then tuned the centre. Cache v101. Local gate suite green. Before and after pixel-change measurements are in the calibration log. Proposal queued for cold read. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Deploying resilience-eval-ai with
|
| Latest commit: |
705b120
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://713dee08.resilience-eval-ai.pages.dev |
| Branch Preview URL: | https://garden-pool-flow-2026-09-25.resilience-eval-ai.pages.dev |
Adversarial review found that the pool ripple noise was sampled in a basis that rotated from pixel to pixel, which aliased the ripples into grain across the pool centre; an independent port confirmed about four in ten pool pixels sampled above the Nyquist limit. The noise domain now uses one fixed downstream axis while the motion and refraction still follow the local field. The asset record states the trade-off. The calibration entry is corrected: the owner judged the first pass, the splash speed varies fourteen to eighteen, the pixel-difference figures are a session observation with no retained script, the removed drift was about twelve pixels per second, the sampling guard is described exactly, and the upper basin slides leftward. Co-Authored-By: Claude Fable 5.1 <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.
What this changes
The owner reported that the pool the waterfall empties into read as stagnant while the falls poured into it. The pool section of
assets/garden-water-v1.jsis now a flow field:Falls, masks, still image, vertex shader, uniforms and the motion lifecycle (pause, reduced motion, hidden page, tool open, context loss) are unchanged. Speeds are image-space art speeds, not physical water. Cache v101.
Verification
verify-atlas-runtime.jsexpects v101.data-staterunning, frame count advancing, no console errors).Not done here
make-the-pool-flow-downstream-from-the-fall-footis queued for the owner's cold read; merge is not acceptance.🤖 Generated with Claude Code