Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

math processing error on valid asciimathml containing invisible grouping and grouping #747

Closed
ilikerobots opened this issue Feb 20, 2014 · 9 comments

Comments

Projects
None yet
3 participants
@ilikerobots
Copy link

commented Feb 20, 2014

Mathjax is showing a [Math Processing Error] when typesetting some asciimathml formulas containing a grouping nested within an invisible grouping, despite being valid asciimathml. The following examples illustrate:

<ul>
<li>pass:  `root 2 1` </li>
<li>pass:  `root 2 (1)` </li>
<li>pass:  `root 2 {:1:}`</li>
<li>fail:  `root 2 {:(1):}`</li>
</ul>
<ul>
<li>pass:  `1^2`</li>
<li>pass:  `1^[2]`</li>
<li>pass:  `1^{:2:}`</li>
<li>fail:  `1^{:[2]:}`</li>
</ul>
<ul>
<li>pass:  `1/2`</li>
<li>pass:  `1/{2}`</li>
<li>pass:  `1/{:2:}`</li>
<li>fail:  `1/{:{2}:}`</li>
</ul>
@pkra

This comment has been minimized.

Copy link
Member

commented Feb 20, 2014

Thanks for reporting this.

@pkra pkra added Accepted and removed Accepted labels Feb 20, 2014

@pkra

This comment has been minimized.

Copy link
Member

commented Feb 20, 2014

Just a note. Pure asciimathml renders the "failed" ones not like the other entries in these lists -- the inner grouping brackets are not considered grouping.

@ilikerobots

This comment has been minimized.

Copy link
Author

commented Feb 20, 2014

Yes, correct, I wasn't clear. The failed ones weren't necessarily intended to be identical to the passing ones. I was intending to indicate that either type of grouping worked individually, but that Mathjax was unable to typset when both were together.

Here's the asciimathml rendering from chapman.edu/~jipsen, just to be clear:
screenshot_mathjax_747

Thanks!

@pkra

This comment has been minimized.

Copy link
Member

commented Feb 20, 2014

Thanks for the confirmation.

@dpvc

This comment has been minimized.

Copy link
Member

commented Feb 20, 2014

It turns out that this is due to a problem with the handling of an mrow's nodeValue in the code that implements the DOM calls in the glue that hooks the AsciiMathML code into MathJax's internal MathML representation. Easily fixed.

@dpvc dpvc added this to the Bugfix Version milestone Feb 20, 2014

@ilikerobots

This comment has been minimized.

Copy link
Author

commented Feb 21, 2014

Confirmed this fixes the issue. Thanks for the quick fix.

@dpvc

This comment has been minimized.

Copy link
Member

commented Feb 21, 2014

Glad it worked for you! Thanks for the confirmation.

dpvc pushed a commit to dpvc/MathJax that referenced this issue Feb 21, 2014

@dpvc dpvc added Merged and removed Ready for Review labels Feb 21, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Feb 21, 2014

=> Merged.

@dpvc dpvc closed this Feb 21, 2014

dpvc pushed a commit to mathjax/MathJax-test that referenced this issue Apr 20, 2014

@dpvc

This comment has been minimized.

Copy link
Member

commented Apr 20, 2014

=> In Test Suite

AsciiMathToMathML/issue747.html

@dpvc dpvc added v2.4 and removed Merged labels Jun 30, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.