Tote Stellen: sim.trail, flyToApex, scene.background gegen setClearColor #6

New issue

Open

opened 2026-10-18 23:23:34 +02:00 by gilli · 0 comments

gilli commented

2026-10-18 23:23:34 +02:00

Owner

Problem

Drei Stellen erzeugen Arbeit, die niemand abnimmt.

1. sim.trail wird nie gelesen. src/physics.js:365-370 schiebt alle 0.2 s einen Punkt und beschneidet bei 4000 Einträgen. Kein Renderer, Test oder Tool verweist auf trail (im ganzen Repo außerhalb von physics.js geprüft).

2. flyToApex hat keinen Aufrufer. Exportiert in src/physics.js:542, im Repo nirgends importiert. simulate() deckt denselben Ablauf ab und wird von tools/check.mjs und tools/fly.mjs benutzt.

3. scene.background gewinnt gegen setClearColor. src/render.js:216 setzt scene.background, src/render.js:502 setzt pro Frame in follow() den Clear-Color aus der Höhenfarbe. three.js zeichnet die Hintergrundfarbe der Szene über den Clear-Color, d. h. der in follow() berechnete Höhen-Farbumschlag wird verworfen.

Vorschlag

1 und 2 löschen. Bei 3 entscheiden, was gemeint war: entweder scene.background entfernen und setClearColor wirkt, oder setClearColor durch this.scene.background.copy(c) ersetzen.

Prüfung

npm test grün; bei 3 ein Headless-Screenshot bei 0 m und bei 60 km mit sichtbar anderem Himmelston, wenn der Umschlag gewollt ist.

## Problem Drei Stellen erzeugen Arbeit, die niemand abnimmt. **1. `sim.trail` wird nie gelesen.** `src/physics.js:365-370` schiebt alle 0.2 s einen Punkt und beschneidet bei 4000 Einträgen. Kein Renderer, Test oder Tool verweist auf `trail` (im ganzen Repo außerhalb von `physics.js` geprüft). **2. `flyToApex` hat keinen Aufrufer.** Exportiert in `src/physics.js:542`, im Repo nirgends importiert. `simulate()` deckt denselben Ablauf ab und wird von `tools/check.mjs` und `tools/fly.mjs` benutzt. **3. `scene.background` gewinnt gegen `setClearColor`.** `src/render.js:216` setzt `scene.background`, `src/render.js:502` setzt pro Frame in `follow()` den Clear-Color aus der Höhenfarbe. three.js zeichnet die Hintergrundfarbe der Szene über den Clear-Color, d. h. der in `follow()` berechnete Höhen-Farbumschlag wird verworfen. ## Vorschlag 1 und 2 löschen. Bei 3 entscheiden, was gemeint war: entweder `scene.background` entfernen und `setClearColor` wirkt, oder `setClearColor` durch `this.scene.background.copy(c)` ersetzen. ## Prüfung `npm test` grün; bei 3 ein Headless-Screenshot bei 0 m und bei 60 km mit sichtbar anderem Himmelston, wenn der Umschlag gewollt ist.

gilli added the

cleanup

label

2026-10-18 23:23:34 +02:00