Skip to content

[Test]: Add test suite for Llama4 RoPE implementation#1004

Merged
Tcc0403 merged 1 commit intolinkedin:mainfrom
noemotiovon:op_llama4_rope
Jan 12, 2026
Merged

[Test]: Add test suite for Llama4 RoPE implementation#1004
Tcc0403 merged 1 commit intolinkedin:mainfrom
noemotiovon:op_llama4_rope

Conversation

@noemotiovon
Copy link
Contributor

  • Add test_llama4_rope.py
  • Test correctness against HuggingFace implementation (forward & backward)
  • Test functional vs class interface consistency
  • Support multiple configurations: different batch sizes, sequence lengths, head dimensions, and q/kv head ratios
  • Support both float32 and bfloat16 dtypes
  • Follow existing test patterns from test_rope.py

Hardware Type: NVIDIA A100-SXM4-80GB

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

- Add test_llama4_rope.py with comprehensive test coverage
- Test correctness against HuggingFace implementation (forward & backward)
- Test functional vs class interface consistency
- Support multiple configurations: different batch sizes, sequence lengths,
  head dimensions, and q/kv head ratios
- Support both float32 and bfloat16 dtypes
- Follow existing test patterns from test_rope.py
@noemotiovon
Copy link
Contributor Author

Test Result:
image

Copy link
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.

LGTM, thank you!

@Tcc0403 Tcc0403 merged commit 97600cd into linkedin:main Jan 12, 2026
3 of 7 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.

2 participants