-
Notifications
You must be signed in to change notification settings - Fork 19
add testmath as amsmath test file #918
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we are in the process of setting up auto testing of all test files you might consider keeping failures as part of the test. Then corrections like a new luamml will generate a diff when an issue is resolved (but I don't know how long it will take to see this being activated).
With the development version of tagpdf and luamml I opened an issue about the crash #921 |
Okay I added back the smallmatrix envs |
We just merged the change to do automated testing, so it would be awesome if this MR could be rebased and adjusted:
|
@zauguin Which directory do I run the l3build command in? Also, is that the right syntax? When I do that it says the usage is wrong |
l3build save -c ... I guess. And in the dir where the config file is. |
Okay should be good now |
@mbertucci47 Thanks. Could you also move the |
@zauguin I moved the xml files. Is there any way to make that automatic when l3build is run? |
Uses the file testmath.tex as a test file for amsmath. So it didn't error, I made the adjustments
P_\X
toP_{\X}
(for unicode-math)\@makecol
smallmatrix
envs (smallmatrix
luamml issue #875)I tried running it with
tagging-setup={math/setup=mathml-SE}
as well but this results in a lot of warnings. When I tried to isolate the warnings in a MWE, they went away so I gave up. It is also somewhat difficult to debug because either the line numbers in the warning seem wrong or there are no line numbers in the warning.