Skip to content

Add NaN handling to KumoAnomaly - #1235

Merged
rusty1s merged 8 commits into
mainfrom
anomaly_nan_handling
Oct 22, 2026
Merged

rusty1s merged 8 commits into
mainfrom
anomaly_nan_handling

Conversation

@rusty1s

@rusty1s rusty1s commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

aw471 and others added 7 commits October 21, 2026 09:43
Co-authored-by: Aditi Gautam <adgautam@nvidia.com>
Add optional `attn_residual_norm` and `mlp_residual_norm` modules to `TransformerBlock`, and apply its dropout to the attention output before the residual addition. Build the Kumo-Anomaly `ResidualBlock` on a new `KumoAnomalyTransformerBlock` instead of `torch.nn.TransformerEncoderLayer`.
Co-authored-by: Aditi Gautam <adgautam@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Oct 22, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@rusty1s
rusty1s merged commit 625a15f into main Oct 22, 2026
@rusty1s
rusty1s deleted the anomaly_nan_handling branch October 22, 2026 10:10
@codecov

codecov Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.86%. Comparing base (e45491f) to head (532ffdc).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1235   +/-   ##
=======================================
  Coverage   82.86%   82.86%           
=======================================
  Files         173      173           
  Lines       11136    11136           
=======================================
  Hits         9228     9228           
  Misses       1908     1908           

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

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