Commit cf8207f
Remove L* tone reversal analysis (Analysis tab + iccviz)
Drops the L* tone-reversal analysis from the profiletool Analysis tab and the
iccviz IccVizModel engine, at the user's request. The Analysis tab now hosts
Profile Statistics and Neutral Axis Inking only.
Frontend:
- AnalysisPanel.jsx: remove the ReversalSection / ReversalGraphs / ReversalPlot
and their helpers (decodeSegments, inkTransition, formatWarnings, VizWarnings,
the reversal cache) and reversal state; drop the now-unused useRef import.
- AnalysisPanel.module.css: prune the reversal-only styles (custom SVG plot,
subplot grid, colour-filter chips, ranked-table cells, warnings).
- i18n.jsx: remove the 12 reversal-only keys across all 12 locales; regenerate
the translation xlsx (sync-translations.mjs) — 187 keys.
- ProfileViewer.jsx: update the Analysis-tab comment.
iccviz (C++ engine):
- IccVizModel.hpp: retire Kind::InkReversalL = 7 (commented out, value reserved
so the append-only enum keeps NeutralAxisInking = 8).
- IccVizModel.cpp: remove the reversal algorithm section, its Enumerate()
emission, and the RenderGraph() dispatch (308 lines). The shared helpers
kMaxInkChannels / isPcsSpace / pcsToLstar are kept (used by the neutral-axis,
gamut-volume, and round-trip paths).
- Readme.md: update the enum doc.
User guide: MANUAL.md gains a "3.4 Analysis" section documenting Profile
Statistics + Neutral Axis Inking (the Analysis tab was previously undocumented);
XML/JSON renumbered to 3.5/3.6; help.html regenerated.
The WASM artifacts are intentionally NOT rebuilt here — mirroring the prior
iccviz change (33e4aac), they are regenerated and committed at release time.
A full clean rebuild against iccDEV origin/master was verified locally: the C++
compiles, iccplot.wasm shrinks ~24 KB, and all reversal strings are gone.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XTfq9wJh9ZWT1fTN62LkDN1 parent 33e4aac commit cf8207f
18 files changed
Lines changed: 53 additions & 906 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
29 | | - | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
30 | 31 | | |
31 | 32 | | |
32 | 33 | | |
| |||
135 | 136 | | |
136 | 137 | | |
137 | 138 | | |
138 | | - | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
139 | 157 | | |
140 | 158 | | |
141 | 159 | | |
142 | 160 | | |
143 | 161 | | |
144 | | - | |
| 162 | + | |
145 | 163 | | |
146 | 164 | | |
147 | 165 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
266 | 266 | | |
267 | 267 | | |
268 | 268 | | |
269 | | - | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
270 | 281 | | |
271 | 282 | | |
272 | | - | |
| 283 | + | |
273 | 284 | | |
274 | 285 | | |
275 | 286 | | |
| |||
0 commit comments