Commit 21b61eb
committed
docs(pr-x12): address PR #197 CodeRabbit review — 7 open items
Arithmetic errors
- R-11 (canon-resolutions): 132,710 was 8×8 leaves/frame, not 64×64
CTUs (which is 2,040). Relabelled the per-CTU breakdown table and
budget derivation to per-leaf; clarified the 1 CTU = ~64 leaves at
max split depth relationship. Latency narrative unchanged.
- R-12 (canon-resolutions): per-CTU flush rate at 4K was 80,000/sec
but 2,040 CTUs × 60 fps = 122,400/sec; 1080p figure refined to
30,600/sec (510 CTUs × 60).
- x266 §6 bandwidth: HEVC at 4K/60 from 1080p/30 scales as
4 (res) × 2 (fps) × 6.25 MB = 50 MB, not 4 × 4 × 6.25 = 100 MB.
PR-X12 wins by ~6× (not 12×); crossover ~1.3× (not 3×).
Bits 14-15 consistency (3 sites reconciled to R-2 canon)
- merged-canon M:E-J §3 formalisation note: replaced stale
'bits 14-15 = leaf_size' with R-2's bit 15 = UNIVERSAL inter-tier /
bit 14 = CONSUMER-TYPED via ConsumerProfile. Leaf size lives in
Ctu<const N> at the type level (M:E-G), not in header bits.
- x266 §5 header layout: same R-2 alignment for the HEVC-compatible
header block.
- anti-neural §6: enhancement-layer flag moved from per-leaf bit 14
(claimed by R-2 consumer-typed demux) to the frame header reserved
area (alongside ConsumerProfile and FlushUnit per R-2/R-12).
Path / citation corrections (cam-pq-sigker-dn-tree doc)
- Absolute /home/user/ndarray/... paths replaced with repo-relative
src/hpc/... since this repo IS ndarray.
- /home/user/lance-graph/... paths marked as external repo
(adaworldapi/lance-graph), not resolvable in this checkout.
- Hambly-Lyons 2010 citation now includes arXiv:math/0507536 and
Annals of Mathematics 171(1):109-167.
All claims re-verified by reading each affected file in full
(canon-resolutions 1281 lines, x266 321, merged-canon 647,
anti-neural 337, cam-pq 395) before edits — the same protocol
applied to the bgz-jc proofread.
https://claude.ai/code/session_01HbqooFZHAjaUtFEzhA1R2u1 parent f1d68e6 commit 21b61eb
5 files changed
Lines changed: 58 additions & 32 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
234 | 234 | | |
235 | 235 | | |
236 | 236 | | |
237 | | - | |
| 237 | + | |
238 | 238 | | |
239 | 239 | | |
240 | 240 | | |
| |||
Lines changed: 15 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| |||
108 | 108 | | |
109 | 109 | | |
110 | 110 | | |
111 | | - | |
| 111 | + | |
112 | 112 | | |
113 | 113 | | |
114 | 114 | | |
| |||
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
137 | | - | |
| 137 | + | |
138 | 138 | | |
139 | 139 | | |
140 | 140 | | |
| |||
206 | 206 | | |
207 | 207 | | |
208 | 208 | | |
209 | | - | |
| 209 | + | |
210 | 210 | | |
211 | 211 | | |
212 | 212 | | |
| |||
222 | 222 | | |
223 | 223 | | |
224 | 224 | | |
225 | | - | |
| 225 | + | |
226 | 226 | | |
227 | 227 | | |
228 | 228 | | |
| |||
376 | 376 | | |
377 | 377 | | |
378 | 378 | | |
379 | | - | |
380 | | - | |
381 | | - | |
382 | | - | |
383 | | - | |
384 | | - | |
385 | | - | |
386 | | - | |
387 | | - | |
| 379 | + | |
| 380 | + | |
| 381 | + | |
| 382 | + | |
| 383 | + | |
| 384 | + | |
| 385 | + | |
| 386 | + | |
| 387 | + | |
| 388 | + | |
388 | 389 | | |
389 | 390 | | |
390 | 391 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
767 | 767 | | |
768 | 768 | | |
769 | 769 | | |
770 | | - | |
771 | | - | |
| 770 | + | |
| 771 | + | |
| 772 | + | |
| 773 | + | |
| 774 | + | |
| 775 | + | |
| 776 | + | |
| 777 | + | |
772 | 778 | | |
773 | 779 | | |
774 | | - | |
| 780 | + | |
775 | 781 | | |
776 | 782 | | |
777 | 783 | | |
| |||
781 | 787 | | |
782 | 788 | | |
783 | 789 | | |
784 | | - | |
| 790 | + | |
785 | 791 | | |
786 | | - | |
| 792 | + | |
787 | 793 | | |
788 | 794 | | |
789 | | - | |
| 795 | + | |
790 | 796 | | |
791 | 797 | | |
792 | 798 | | |
| |||
797 | 803 | | |
798 | 804 | | |
799 | 805 | | |
800 | | - | |
801 | | - | |
| 806 | + | |
| 807 | + | |
802 | 808 | | |
803 | 809 | | |
804 | 810 | | |
| |||
813 | 819 | | |
814 | 820 | | |
815 | 821 | | |
816 | | - | |
| 822 | + | |
| 823 | + | |
817 | 824 | | |
818 | 825 | | |
819 | 826 | | |
820 | 827 | | |
821 | | - | |
822 | | - | |
| 828 | + | |
| 829 | + | |
823 | 830 | | |
824 | 831 | | |
825 | 832 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
278 | 278 | | |
279 | 279 | | |
280 | 280 | | |
281 | | - | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
282 | 293 | | |
283 | 294 | | |
284 | 295 | | |
| |||
Lines changed: 12 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
163 | 163 | | |
164 | 164 | | |
165 | 165 | | |
166 | | - | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
167 | 173 | | |
168 | 174 | | |
169 | 175 | | |
| |||
214 | 220 | | |
215 | 221 | | |
216 | 222 | | |
217 | | - | |
218 | | - | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
219 | 226 | | |
220 | 227 | | |
221 | 228 | | |
222 | | - | |
| 229 | + | |
223 | 230 | | |
224 | 231 | | |
225 | 232 | | |
226 | | - | |
| 233 | + | |
227 | 234 | | |
228 | 235 | | |
229 | 236 | | |
| |||
0 commit comments