Skip to content

Fix algorithm name in dataflow walkthrough - #2

Open
RaviTejGuntuku wants to merge 1 commit into
LonghornSilicon:mainfrom
RaviTejGuntuku:patch-1
Open

Fix algorithm name in dataflow walkthrough#2
RaviTejGuntuku wants to merge 1 commit into
LonghornSilicon:mainfrom
RaviTejGuntuku:patch-1

Conversation

@RaviTejGuntuku

Copy link
Copy Markdown

the paper that proposed the online softmax algorithm to avoid manifesting the entire O(N^2) QK^T matrix was the ORIGINAL FlashAttention paper, not FlashAttention3.

the paper that proposed the online softmax algorithm to avoid manifesting the entire O(N^2) QK^T matrix was the ORIGINAL FlashAttention paper, not FlashAttention3.
A14N77 added a commit that referenced this pull request Jun 6, 2026
v0.4.1 — close audit gaps #1/#2/#4 + dataflow doc section
themoddedcube added a commit that referenced this pull request Jul 20, 2026
Re-derive the MatE integer accumulator for Qwen2 dims (was "TBD pending
re-derivation"). The two integer reduction axes need different widths:

- INT24: W4A8 GEMM + Q·Kᵀ (reduce over hidden/head dim ≤4096; 10b product × 4096 = 22b).
- INT32: INT8 P·V (reduces over the TOKEN dim -> width scales with context). Worst-case
  flat attention needs 14+ceil(log2 L) bits; INT24 only covers ~520 tokens, INT32 ~133k.
  Empirically real Qwen2 stays at 21b and doesn't grow with context (peaky softmax), so
  INT24 works in practice, but INT32 is required for guaranteed correctness.
- FP32 for FP16 tiles (unchanged).

Resolves MatE README open-question #2. Evidence:
adaptive-precision-attention/analysis/pv_accumulator_width.py. Updates arch.yml
(pe_op_attention_pv, accumulator_rationale, key_decisions), MatE README, STATUS log.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant