Skip to content

Fix empty array edge cases in nlinfit - #485

Open
Sonu0305 wants to merge 4 commits into
gnu-octave:mainfrom
Sonu0305:fix-nlinfit-empty
Open

Fix empty array edge cases in nlinfit#485
Sonu0305 wants to merge 4 commits into
gnu-octave:mainfrom
Sonu0305:fix-nlinfit-empty

Conversation

@Sonu0305

Copy link
Copy Markdown
Contributor

No description provided.

@pr0m1th3as

Copy link
Copy Markdown
Member

Matches MATLAB, which errors in all these cases. Remove the %!test block that holds only a comment (it always passes), give the %!error tests the full message with the nlinfit: prefix, and start the message after the prefix in lower case, e.g. "nlinfit: MODELFUN must return a vector of the same size as Y."

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