Skip to content

CI, tokenizer fix, and the --bench throughput harness#5

Merged
codewithfourtix merged 4 commits into
mainfrom
ci/github-actions
Jul 14, 2026
Merged

CI, tokenizer fix, and the --bench throughput harness#5
codewithfourtix merged 4 commits into
mainfrom
ci/github-actions

Conversation

@codewithfourtix

Copy link
Copy Markdown
Owner

Completes the project: green CI plus a benchmark harness, so the whole thing is verified to compile, test, and run.

What landed

  • GitHub Actions CIcargo build --release + cargo test on every push (Ubuntu runner). All 13 kernel unit tests pass; the release binary builds clean.
  • Tokenizer build fix — use tokenizers default features (onig) so it compiles (needs a C toolchain, present on Linux/macOS/CI).
  • --bench — throughput on a randomly-weighted Qwen2.5-0.5B-shaped model (no weights needed); CI runs it, so the forward pass is exercised end-to-end.

Measured (2-vCPU CI runner)

scheme memory tok/s
f32 1976 MB 7.1
int8 496 MB 4.4
int4 g64 278 MB 2.4

Quantization is a 4x/7x memory win; the scalar dequant trades throughput (SIMD dequant is the next optimization). The engine now compiles, tests green, and runs on real hardware.

Compiles the release binary and runs the kernel unit tests on every push.
Builds a Qwen2.5-0.5B-shaped model with random weights (timing depends on matrix
sizes, not values) and times decode throughput per quant scheme. CI runs it, so
the forward pass is exercised end-to-end on every push.
Copilot AI review requested due to automatic review settings July 14, 2026 08:41
@codewithfourtix
codewithfourtix merged commit f998a73 into main Jul 14, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codewithfourtix
codewithfourtix deleted the ci/github-actions branch July 14, 2026 08:42
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.

2 participants