Skip to content

Fix empty array edge cases in mnrfit - #487

Open
Sonu0305 wants to merge 2 commits into
gnu-octave:mainfrom
Sonu0305:fix-mnrfit-empty
Open

Sonu0305 wants to merge 2 commits into
gnu-octave:mainfrom
Sonu0305:fix-mnrfit-empty

Conversation

@Sonu0305

Copy link
Copy Markdown
Contributor

Fixed an edge case where an empty/invalid categorical structure caused an obscure scalar assignment crash (reshape: SIZE must be non-negative).

@pr0m1th3as

Copy link
Copy Markdown
Member

Message matches MATLAB exactly, no need to replicate exact wording. Remove the %!test block that holds only a comment, restore the blank line before the new check, and drop the extra blank line at the end of the file.

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