rsmenon / MATLink

Communicate with MATLAB from Mathematica
matlink.org
67 stars 14 forks source link

help command returns HTML in MEvaluate #29

Open szhorvat opened 11 years ago

szhorvat commented 11 years ago

Since changing MEvaluate to use evalc, the help command returns HTML.

Example:

In[15]:= MEvaluate["help pi"]

Out[15]= " <strong>pi</strong>     3.1415926535897....
    <strong>pi</strong> = 4*atan(1) = imag(log(-1)) = 3.1415926535897....

    Reference page in Help browser
       <a href=\"matlab:doc pi\">doc pi</a>

"

This can be reproduced in MATLAB by running it as matlab -nodesktop then comparing help pi with evalc('help pi')

szhorvat commented 11 years ago

We have two options here:

The MATLAB GUI does the latter:

Screen Shot 2013-04-11 at 11 00 29

szhorvat commented 11 years ago

Related info:

rsmenon commented 11 years ago

It's not just for help; HTML strings are returned for most errors that include the function name (with a link to the help docs or to the file itself, if it's a custom file). I've reverted MEvaluate, so this issue isn't there right now, but since the plan is to eventually use evalc for MEX, this needs to be figured out.

amroamroamro commented 11 years ago

There is an undocumented way to explicitly turn off hyperlinks and other HTML goodies:

feature('hotlinks','off')

This value is queried internally by help to decide whether to add HTML tags to the output.

Note: I did not test the above..

szhorvat commented 11 years ago

Wow, this is very useful information (which support did not give me). It does indeed work, but I need to use 1 and 0 instead of 'off'. However, it doesn't seem to be possible to easily turn it off for good inside evalc. It is possible to turn it off for a single evalc call only though. Here's a command line transcript showing the behaviour:

>> feature('hotlinks')

ans =

     0

>> evalc('feature(''hotlinks'')')

ans =

ans =

     1

>> evalc('feature(''hotlinks'',0)')

ans =

     ''

>> evalc('feature(''hotlinks'')')  

ans =

ans =

     1

>> evalc('feature(''hotlinks'',0); feature(''hotlinks'')')

ans =

ans =

     0

>> 

Since we're already wrapping everything MEvaluateed with some extra code to catch errors, we might as well add feature('hotlinks',0) at the beginning.

rsmenon commented 11 years ago

@amroamroamro Thanks a lot! That's very useful and saves us the trouble of having to strip HTML tags from the output (which is always messy).

amroamroamro commented 11 years ago

@szhorvat: hmm you are right, for some reason the value seems to be reset inside evalc... Fortunately this only affect help output, error messages do not contain "jump to line" links, even when using evalc.

Try replacing help with the lower level builtin function helpfunc

@rsmenon: come to think of it, stripping tags might not be so bad here. I know one should never parse HTML using regexp, but the set of HTML that is outputted is fairly simple of the form:

<tag attrib="val">text</tag>

Should be easy to strip HTML and extract the text.

amroamroamro commented 11 years ago

Not exactly related to the issue, but you mentioned wrapping evaluated strings in extra code to catch errors.

Now I haven't looked at the source code yet, but let me share another less-known trick; eval and evalc take a second input that is executed only when the evaluation errors in the first:

isError = false;
evalc('nonexistent', 'isError = true;')

maybe this pattern could be used in your code..

szhorvat commented 11 years ago

@amroamroamro Thanks! This does look very useful indeed. This will all become very relevant when we switch to the MEX interface (instead of the Engine interface). Perhaps we could have a chat sometime about how we're planning to do this. I'd love to listen to your opinion! (freenode or SO chat or something else)

amroamroamro commented 11 years ago

@szhorvat sure i just joined ##matlab on freenode. you'll find me on the top the list :)

szhorvat commented 10 years ago

With work having started on the MEX version, this issue is now more serious. It's most annoying in error messages.

Observation:

Right now we use getReport() to extract the message string. If MATLAB is run as matlab -nodesktop, it getReport() won't return any HTML in the message string.

However, help still does.

amroamroamro commented 10 years ago

getReport() has an explicit option to turn off hyperlinks, regardless of how MATLAB was started:

try
    %...
catch ME
    errmsg = getReport(ME, 'extended', 'hyperlinks','off');
    disp(errmsg)
end