Skip to content

Add behavioral tests for the NanoTabICL guide examples - #1237

Merged
rusty1s merged 7 commits into
mainfrom
coderabbit/add-pull-request-tests/f72b503c
Oct 24, 2026
Merged

rusty1s merged 7 commits into
mainfrom
coderabbit/add-pull-request-tests/f72b503c

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Add regression coverage for the executable NanoTabICL guide examples and their model components.

Description of the change

Execute the guide's Python blocks directly to keep tests aligned with the published examples. Cover context and query isolation, permutation behavior, readout shapes, both context-label injection stages, invalid labels, empty queries, and feature-grouping offsets, wraparound, projection, and input dimensions. Parameterize forward checks across float32 and float64 with CPU and CUDA coverage, and initialize attention outputs so behavioral checks exercise attention.

Validation

Validation was not run; changes were inspected only.

View coding task

@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.

@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 83.02%. Comparing base (01d13c9) to head (8ac32ed).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1237   +/-   ##
=======================================
  Coverage   83.02%   83.02%           
=======================================
  Files         174      174           
  Lines       11324    11324           
=======================================
  Hits         9402     9402           
  Misses       1922     1922           

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

Base automatically changed from aaron/nanotabicl-guide to main October 24, 2026 09:31
@rusty1s
rusty1s merged commit 22d731e into main Oct 24, 2026
5 checks passed
@rusty1s
rusty1s deleted the coderabbit/add-pull-request-tests/f72b503c branch October 24, 2026 09:47
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