From 94c3eacb726181a35294a36f18d32e7a1d919d53 Mon Sep 17 00:00:00 2001
From: maj00r <maj00r@host>
Date: Wed, 14 Oct 2026 22:28:52 +0200
Subject: [PATCH] fix(tools): water keeps its large param_specular as the glint
 of the sun on its fixed mirror and is only reported as information; the norm
 says which files are checked, that textures without a material go on their
 diffuse alone, and that diffuse, specgloss and normal maps all become BC7
 made from the TGA

---
 tools/materials/README.md             | 4 +++-
 tools/materials/validate_materials.py | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/tools/materials/README.md b/tools/materials/README.md
index 8009a923..2a6ce3b3 100644
--- a/tools/materials/README.md
+++ b/tools/materials/README.md
@@ -80,6 +80,8 @@ python3 tools/materials/validate_materials.py ~/pctga dynamic --fallback <katalo
 | `--info` | pokazuje też informacje, nie tylko błędy i uwagi |
 | `-j N` | liczba procesów, domyślnie wszystkie rdzenie; cały pctga trwa ok. 80 s |
 
+Sprawdza pliki `.mat` (w pctga są w `dynamic` i `textures`); tekstury bez `.mat` idą na samym diffuse i nie ma czego w nich sprawdzać. Woda (`shader: water`) ma stałe lustro, jej duży `param_specular` daje odblask słońca i jest tylko informacją.
+
 Wynik: dla każdego materiału z problemem ścieżka `.mat`, shader i lista komunikatów, na końcu podsumowanie.
 
 - `BLAD` – trzeba poprawić (np. połysk niemetalu kilkanaście razy za mocny, brak tekstury, nieistniejący shader).
@@ -93,4 +95,4 @@ Walidator czyta pliki shaderów i z nich rozpoznaje, co każdy shader robi ze sp
 1. Uruchom walidator na swoim katalogu pojazdu, np. `dynamic/pkp/sa103`.
 2. Najpierw `BLAD`: zwykle R w specgloss (niemetal ok. 15/255) albo `param_specular` (zostaw 1, siłę połysku daje R).
 3. Potem `UWAGA`: G (gładkość) według tabeli wartości docelowych, B tylko 0 albo 255, metal z jasnym diffuse.
-4. Poprawiaj źródłowe TGA i uruchom walidator jeszcze raz. Z TGA będą robione tekstury BC7, które zastąpią obecne DDS, więc poprawki w DDS przepadną.
+4. Poprawiaj źródłowe TGA i uruchom walidator jeszcze raz. Z TGA będą robione tekstury BC7 (diffuse, specgloss i normalmapy), które zastąpią obecne DDS, więc poprawki w DDS przepadną. BC7 nie ma bloków DXT, więc szum widoczny w specgloss i alfie normalmapy to szum z TGA, nie z kompresji.
diff --git a/tools/materials/validate_materials.py b/tools/materials/validate_materials.py
index 53618865..fa6e360f 100644
--- a/tools/materials/validate_materials.py
+++ b/tools/materials/validate_materials.py
@@ -253,7 +253,9 @@ def check(mat, values, nested, shaders, resolver, images, rel):
 
     if ps is None and shader['spec'] != 'none':
         add(INFO, 'param_specular nie ustawiony, bierze jasnosc specular submodelu z e3d/t3d')
-    if ps is not None and ps > PARAM_SPECULAR_MAX:
+    if ps is not None and ps > PARAM_SPECULAR_MAX and shadername.lower().startswith('water'):
+        add(INFO, 'param_specular %g w shaderze %s: odblask slonca na lustrze wody, poza norma' % (ps, shadername))
+    elif ps is not None and ps > PARAM_SPECULAR_MAX:
         add(ERROR if ps > 2 * PARAM_SPECULAR_MAX else WARN,
             'param_specular %g mnozy polysk %gx (norma 1, sile niesie R; w starszych shaderach rozjasnia tez w reflektorach)' % (ps, ps))
     psv = ps if ps is not None else 1.0