The rewrite (D-049) starts from a clean slate; this commit is the reference
implementation of the graduated design.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The debug scene had one DirectionalLight3D at identity rotation (shining
horizontally along -Z), no sky, and ambient sourced from the background,
which with no sky is the flat clear colour. Terrain read flat, shadowed
faces crushed, nothing to reflect. We have been judging terrain and water
against that -- a bad-evidence trap.
Minimal and standard, nothing else:
* sun angled to 52 deg elevation, shadows ENABLED, shadow max distance
2500 m (the default 100 m is useless at this world scale)
* WorldEnvironment gains a ProceduralSkyMaterial sky, background_mode Sky
* ambient sourced from that sky at 0.45 energy, so shadow sides read
No post-fx, no volumetrics, no SSAO, no tonemap change -- renderer-chat work.
MEASURED on seed 1280587109, identical camera, before vs after (mean region
colour off the rendered frames):
land luma 36 -> 122
mountain luma 76 -> 188, blue-excess -0.3 -> +49.7 (snow now reads as
lit snow catching sky, not flat grey)
TERRAIN JUDGEABILITY IS THE WIN, and it is large.
BUT THE TASK'S PREMISE DID NOT HOLD, and it should not be quietly buried:
"the sky is what fixes the [water] grey, not re-tuning the water" -- it
does not. Same measurement, water region:
water luma 61 -> 116, blue-excess -3.7 -> -6.2 (BRIGHTER, and
LESS blue -- it reads as bright grey-brown, not as water)
Mechanism: the water material is non-metallic at roughness 0.35 with
VertexColorUseAsAlbedo and alpha 0.62 in the shallows. A rough dielectric
barely shows a sky reflection; what you actually see is ~62 % water tint
over ~38 % seabed, and the seabed is exactly what the new ambient just
brightened. Lighting the world lights what shows THROUGH the water more
than it lights the water.
The fix is a water-material dial -- WATER_ALPHA_SHALLOW/DEEP, or giving the
surface enough specular to return the new sky -- and this task is forbidden
from touching task 13's material, correctly, since that is a separate
judgement. Left for the developer; reported rather than silently worked
around.
The camera's colinear LookAt warning is UNCHANGED and still present. It
comes from cam.LookAt() pointing straight down in ServerChunkManager, not
from the light, so angling the sun could never have cleared it. Verified.
Scene file only -- no code changed.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
