Fix anova2 empty input handling - #499
sahilphad07-sudo wants to merge 2 commits into
Conversation
|
Thank you for looking into this, but I am closing this without merging. Returning A kind request for future contributions: please test the whole surface a change touches before opening a PR, every output and the neighbouring cases, not only the one reported. A partial fix costs more review time than it saves, and we would much rather receive fewer PRs that are complete. |
|
Thank you for the detailed feedback, @pr0m1th3as From next time, I’ll make sure to test the full behavior surface, including neighboring and edge cases, before opening future PRs. I appreciate the guidance. |
anova2failed with an undefinedRIdxerror when the inputxwas empty.This change adds handling for the empty-input case and returns
[NaN, NaN], matching MATLAB behavior.Before
After
MATLAB Behavior
Regression Test Added
Test Results
All existing and new
anova2BISTs pass: