Skip to content

remove pointless warning - #75

Merged
siko1056 merged 1 commit into
sqlp:masterfrom
araujoms:patch-1
Mar 13, 2023
Merged

remove pointless warning#75
siko1056 merged 1 commit into
sqlp:masterfrom
araujoms:patch-1

Conversation

@araujoms

@araujoms araujoms commented Mar 1, 2023

Copy link
Copy Markdown
Contributor

It doesn't make sense to use struct('issym',true)) here as XX is just a numerical matrix. It was causing pointless warnings.

It doesn't make sense to use `struct('issym',true))` here as `XX` is just a numerical matrix. It was causing pointless warnings.
@siko1056

siko1056 commented Mar 12, 2023

Copy link
Copy Markdown
Member

In line 92 before that code I see

XX = XX + XX';

Doesn't this make the input of eigs a symmetric matrix? What is the warning seen in Matlab and what is the warning exactly?

@araujoms

Copy link
Copy Markdown
Contributor Author

Yes, XX is symmetric, but that's not the issue. The warning I get from MATLAB is "Warning: Ignoring issym field in the options structure since the first input is not a function handle." It's because this option only applies when XX is not a numerical matrix, but a function handle. See the documentation. And in SeDuMi's code XX is always just a numerical matrix as far as I can see.

@siko1056 siko1056 mentioned this pull request Mar 13, 2023
@siko1056
siko1056 merged commit 5574f42 into sqlp:master Mar 13, 2023
@araujoms
araujoms deleted the patch-1 branch March 15, 2023 08:36
@siko1056

Copy link
Copy Markdown
Member

Oh my comment was not posted πŸ˜“ Many thanks for the clarification, I merged the PR.

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