isrow.m about python-matlab-bridge HOT 7 CLOSED

How about we ship our own?

http://stackoverflow.com/questions/14933237/which-version-of-matlab-introduced-isrow-and-iscolumn-functions

On Mon, Aug 18, 2014 at 4:30 PM, eendebakpt [email protected]
wrote:

The function isrow is not available on older Matlab versions.

—
Reply to this email directly or view it on GitHub
#81.

from python-matlab-bridge.

FYI, this results in the following warning:

    [Warning: Function isrow has the same name as a MATLAB builtin. We suggest you rename the function to avoid a potential name conflict.]
    [> In path at 110
    In addpath at 87
    In pymat_eval at 38
    In matlabserver at 27]

from python-matlab-bridge.

Hmm. Any suggestions how to deal with this? isrow was apparently introduced in 2011, so we could require that as a minimum, but this warning seems like a small price to pay.

from python-matlab-bridge.

This might be kind of clunky, but you could add it to the path conditionally. Right now everything under matlab/ is added to the path, but you could have a directory that you only add if exist('isrow') is not 0 (and maybe generalize the logic for different builtins that you might decide to ship).

from python-matlab-bridge.

You could suppress warnings while adding that path: warning('off','all').

from python-matlab-bridge.

I think that's probably a better way than selectively adding different
paths. Would you mind making a PR with that?

On Fri, Oct 31, 2014 at 11:23 AM, Steven Silvester <[email protected]

wrote:

You could suppress warnings while adding that path: warning('off','all').

—
Reply to this email directly or view it on GitHub
#81 (comment)
.

from python-matlab-bridge.

Closed by #93.

from python-matlab-bridge.