Skip to content

Architecture diagram shows legacy Post-Norm but includes modern GQA/GLU #104

Description

@Kriston-12

Hi team, thanks for the great resource!

I noticed a slight inconsistency in the main Transformer block diagram in the "All the Transformer Math" chapter.

The diagram currently illustrates a Post-Norm setup (the norm D blocks are placed directly on the main residual stream after the + operations). However, it also explicitly features modern components like GQA and GLU variants.

Since modern LLMs utilizing GQA/GLU (like Llama) almost exclusively use Pre-Norm to maintain a clean residual stream, the norm D blocks should probably be moved to the beginning of the Attention and MLP branches instead.

Thought I'd point this out in case you want to update the diagram to reflect modern topologies!

Image

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions