Clean Dead Grid Shader Code and Restore Probe Split Metrics #17

Open
opened 2026-03-02 17:05:58 +00:00 by ImBenji · 1 comment
Owner

Description: Remove unused shader helpers left after refactor and restore probe visibility for grid vs watermark cost (without changing current visual behavior).

Acceptance: No unused-element warnings in touched files; probe still distinguishes relevant background costs.

Description: Remove unused shader helpers left after refactor and restore probe visibility for grid vs watermark cost (without changing current visual behavior). Acceptance: No unused-element warnings in touched files; probe still distinguishes relevant background costs.
ImBenji added the enhancementarea:renderingarea:performance labels 2026-03-02 17:05:58 +00:00
ImBenji added this to the Arcs & Angles project 2026-03-02 17:08:27 +00:00
ImBenji moved this to In Progress in Arcs & Angles on 2026-03-17 03:55:15 +00:00
Author
Owner

First half done on development (2026-09-05, uncommitted at time of writing). Second half still open, so leaving this issue open on the reduced scope.

Dead code removed. _screenRepeatShaderMatrix is gone from background.dart - it had zero callers and was the last survivor of the rasterised-tile grid approach the fragment shader replaced.

Went further than the title while in there, since it was the same class of thing in the same paint path. Removed from canvas_painter.dart:

  • _v2LineTileImages / _v2UnderlayTileImages / _v2StationTopTileImages and their three *InFlight sets - all declared, cleared in the reset path, never populated or read
    • _v2StaticCompositeKey / _v2StaticCompositePicture / _v2StaticCompositeGuardRect - same shape
      _qtTileImages is NOT dead and was left alone; that is the live qt pyramid cache. flutter analyze clean, full suite green.

Still open: restore the probe split metrics. Grid vs watermark cost is still not separable in the probe output, which is the more useful half of this issue and the reason it stays open. Related to the perf HUD trustworthiness question filed today.

First half done on development (2026-09-05, uncommitted at time of writing). Second half still open, so leaving this issue open on the reduced scope. **Dead code removed.** `_screenRepeatShaderMatrix` is gone from `background.dart` - it had zero callers and was the last survivor of the rasterised-tile grid approach the fragment shader replaced. Went further than the title while in there, since it was the same class of thing in the same paint path. Removed from `canvas_painter.dart`: - `_v2LineTileImages` / `_v2UnderlayTileImages` / `_v2StationTopTileImages` and their three `*InFlight` sets - all declared, cleared in the reset path, never populated or read - - `_v2StaticCompositeKey` / `_v2StaticCompositePicture` / `_v2StaticCompositeGuardRect` - same shape `_qtTileImages` is NOT dead and was left alone; that is the live qt pyramid cache. `flutter analyze` clean, full suite green. **Still open: restore the probe split metrics.** Grid vs watermark cost is still not separable in the probe output, which is the more useful half of this issue and the reason it stays open. Related to the perf HUD trustworthiness question filed today.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: IMBENJI.NET/Metro-Map-Maker#17