Visual: add procedural biome grass and wind#55
Closed
ael-dev3 wants to merge 2 commits into
Closed
Conversation
Owner
Author
|
Superseded by the audited Alpha 0.3.11 integration in #64, now merged to protected main. Accepted work was reconciled, corrected where needed, and validated there; this original branch is intentionally not merged again. |
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.
Scope and branch boundary
mainat3ca99d2d263453fbb112a7a21fa1bfde294e186b(Alpha 0.3.8).341cecb89032e42f62e806f0e8394ba2147833dc.main. No current PR is merged or retargeted here.What changed
green-tuft/dry-tuftpath with deterministic procedural, low-poly instanced grass; static stones remain a separate bounded detail family.MeshStandardMaterial.onBeforeCompileGPU-wind path pinned to the Three r185 shader hook with a stable cache key, fail-closed contract fallback, planted root flex, max-sway bounds, and no grass raycasting.Biome policy
Balanced uses 0.62x candidate density and Reduced uses 0.25x, with deterministic rounding.
Fixed runtime ceilings
Wind updates material uniforms only. Focused regression coverage proves no
setMatrixAtorinstanceMatrix.needsUpdatework occurs during wind steps; matrices update only on active-window repack.Canonical 10,000-cell evidence
The new canonical-surface test runs against 10,000 playable / 10,981 rendered cells. At the center High keep window it attests:
d096966756353ad1d09c90c68ef51a23a6348dc8b8105b638a47a624844401c3.Verification
Passed:
tsc -bNot completed locally:
https://warpkeep.comfrom this environment.Non-goals and residual risks
No SpacetimeDB schema/binding/reducer change, gameplay authority/resource/collision/pathfinding change, deploy, Pages publish, backend publish, tag, release/version claim, or merge occurred.
Density and color remain provisional art-direction values; browser thermal behavior needs continued observation. Grass is decorative and does not react physically to units. Future reviewed structures must supply generic exclusion circles. The shader hook remains intentionally coupled to pinned Three r185, and full-Realm overview intentionally omits individual grass.