SBBCornell
commented
4 years ago I tested the updated branch with the new converter version 1.5. It spins up successfully and returns the correct error message when a figure is missing!
David, maybe we can discuss future tests and do a little planning. Im happy to keep testing as needed but feel my feedback will be most helpful with some parameters and specific goals.
This ticket [ ARXIVNG-2968 ] was originally scoped with addressing the problem of submissions that fail but still seem to generate a PDF (bad).
The problem (or one problem) with the compiler service not signaling errors properly was fixed with recent changes to the converter/compilation image. I'm no longer able to generate a PDF given the example submission originally provided to demonstrate the issue.
The "File Not Found" and the "Emergency Stop" errors are currently highlighted in the log body but not extracted and displayed in the log summary section.
In this PR I've added messages for the "File Not Found" and "Emergency Stop" errors to the log summary section. Since these two errors often occur together I only display one of the errors, the "File Not Found" error, when they occur together. We'll need to find another submission with an emergency stop to demonstrate the other message.
I've also improved the error message displayed when a user uploads XeTeX/LuaTeX source ( currently not supported).
XeTeX/LuaTeX:
I expect Shamsi might be the most interested in the visible log highlighting changes. I'm definitely interested in comments and suggestions to improve this feature of the process page.
Note: At the current time the highlighting routine is independent of the process HTML. That is, things discovered in the markup routine can't be communicated effectively to the processing step, such as to change the state of the submission. In the case where a submission compiles successfully, but we discover something really bad in the log highlighting, it would be nice to 'update' the status of the submission to failed status. Ideas for future improvements.