-
Notifications
You must be signed in to change notification settings - Fork 24
Planetary should do better error reporting for LaTeXML fatal errors (was: silent failure of DruTeXML/LaTeXML for posts containing an unescaped "# symbol") #354
Comments
I would be surprised if the failure is really "silent" - the LaTeXML web service ought to be returning an empty result and a detailed log that should be indicating a Fatal error (as well as the fatal status code - 3). Do you have the returned log somewhere? When I try the code in the web editor, I get exactly what I explain above. There should be better error reporting for fatal errors on the Planetary side, I suspect. P.S. As this is a fatal error in TeX as well, you can't expect a non-empty result here, since the error is rather critical. My suggestion would be to display the log message when the HTML result is empty, to draw attention on the issue. Or some nice-to-the-eye variation on that. |
Yes (and I've changed the issue title to be more general). Just for comparison - https://www.writelatex.com/148379crcbxk - highlights the incorrect line in the input and gives the LaTeX error
This suggests that if we receive a fatal error in LaTeXML we might want to redirect back to the editor page and highlight the line with a comparable message. |
If you want to handle this in the editor, what WriteLaTeX is doing is indeed a good example to follow. The information you would need is already in the log file. |
This showed up again for a user who posted a "bare" URL with a # in it (http://planetmath.org/node/87397). I marked the issue as "priority". |
Adding "editor" tag. Brief discussions by email with @jucovschi @dginev about using WebSockets to help facilitate real-time interaction with the LaTeXML daemon. LaTeX is so flexible that most issues couldn't be solved on a line-by-line basis, but unescaped |
Relevant to "Deyan Ginev peer review metaticket" holtzermann17/planetmath-docs#80 |
Mihnea already has a prototype for reporting LaTeXML errors inline, you should coordinate with him on adopting the code in the main repository. He has a branch for the Planetary development used in the glossary which is KWARC-internal, from what I have understood today. I saw a small demo and it looks nice, but it also hinges on the LaTeXML error reports becoming more precise. We have that as one of the first priorities when we reach 0.9 development, as described at: |
I found him - his GitHub handle is @m-iancu. |
We had discussed this very briefly here: |
This results in a silent failure:
... as "seen" here: http://metameso.org/beta/node/87359 (http://metameso.org/beta/node/87359/source).
[@kohlhase or @jucovschi can you please add Deyan (@dginev) to this repo as a collaborator so I can @-message him about this issue?]
The text was updated successfully, but these errors were encountered: