Skip to content

Conversation

@gramalingam
Copy link
Collaborator

Add RMS Normalization rule variant to support different order of multiplying by scale.

Signed-off-by: Ganesan Ramalingam <[email protected]>
@codecov
Copy link

codecov bot commented Oct 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.39%. Comparing base (75b3d42) to head (82277fd).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2638   +/-   ##
=======================================
  Coverage   70.38%   70.39%           
=======================================
  Files         222      222           
  Lines       26288    26294    +6     
  Branches     2629     2630    +1     
=======================================
+ Hits        18503    18509    +6     
  Misses       6865     6865           
  Partials      920      920           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gramalingam gramalingam enabled auto-merge (squash) October 16, 2025 18:38
Signed-off-by: Ganesan Ramalingam <[email protected]>
@gramalingam gramalingam merged commit 8089bc7 into main Oct 16, 2025
32 checks passed
@gramalingam gramalingam deleted the rama/rmsnorm branch October 16, 2025 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants