Skip to content

[Benchmark] Add ModelConfig Sweep Support and Pre-Probe to Remaining Benchmarks#1195

Merged
Tcc0403 merged 2 commits intolinkedin:mainfrom
lowdy1:left
Apr 18, 2026
Merged

[Benchmark] Add ModelConfig Sweep Support and Pre-Probe to Remaining Benchmarks#1195
Tcc0403 merged 2 commits intolinkedin:mainfrom
lowdy1:left

Conversation

@lowdy1
Copy link
Copy Markdown
Contributor

@lowdy1 lowdy1 commented Apr 17, 2026

Summary

  • Add model_config sweep support to training and Rope op benchmarks, enabling evaluation across different model architectures at a fixed sequence length
  • Introduce a pre-sweep probe step to determine safe configurations and prevent OOM during benchmarking

benchmark_attn_res.py
benchmark_embedding.py
benchmark_fused_neighborhood_attention.py (retains all kernel configs for model_config sweep instead of sweeping model architectures)
benchmark_mhc.py
benchmark_mhc_lm.py
benchmark_multi_token_attention.py
benchmark_relu_squared.py
benchmark_softmax.py
benchmark_sparsemax.py
benchmark_sparse_multi_token_attention.py
benchmark_tiled_mlp.py

  • Hardware Type:
  • run make test to ensure correctness
  • run make checkstyle to ensure code style
  • run make test-convergence to ensure convergence

- Add Qwen 2.5 models (7B / 14B / 72B) and DeepSeek models (V2 Lite / V3) to MODEL_REGISTRY
- Add model_config sweep support to all 33 benchmark scripts, enabling benchmarks to sweep across
  different model architectures at a fixed sequence length
- Refactor benchmark scripts by extracting helper functions:
  - _setup_*
  - _resolve_model_config_*
  to improve code reuse and keep implementations cleaner across sweep modes
- Add grouped bar chart visualization in benchmarks_visualizer for model_config sweep results
Copy link
Copy Markdown
Collaborator

@Tcc0403 Tcc0403 left a comment

Choose a reason for hiding this comment

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

@Tcc0403 Tcc0403 added this pull request to the merge queue Apr 18, 2026
Merged via the queue into linkedin:main with commit 2ca3bd0 Apr 18, 2026
5 checks passed
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.

3 participants