Navigation Menu

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

WIP: Switch to using LaTeXML #210

Merged
merged 67 commits into from Dec 6, 2017
Merged

WIP: Switch to using LaTeXML #210

merged 67 commits into from Dec 6, 2017

Conversation

bfirsh
Copy link
Member

@bfirsh bfirsh commented Nov 29, 2017

This is a proof of concept for switching from Pandoc to LaTeXML to see if it's a good idea. So far, it's looking more promising than our Pandoc hacks. See also #159.

My rough todo list:

  • Switch out pandoc with latexml and see what happens (it works!)
  • Get the postprocessor to convert latexml output into what Distill expects, and style misc things
  • Get integration test suite working
  • Compare output with loads of docs, particular ones that are known not to work with pandoc (failed renders on arxiv-vanity.com, reported bugs, etc)
  • Linkify URLs (probably with https://github.com/sindresorhus/linkify-urls because the regex we use in the Pandoc filter doesn't work in JS).
  • Add http:// to links that don't have it
  • Align math labels correct (probably by putting it in the latex like we did for pandoc instead of using a table)
  • Put table captions on bottom if they're on top (e.g. http://localhost:8010/html/1707.08945v1/ )
  • Get some common publication styles working properly (ICML and IEEE are the ones we have explicit integration tests for)
  • Don't allow resizing of text where we don't want it (e.g. headings, figure captions, bibliography items, etc)
  • Get h5 and h6 working again (just commented out in headings postprocessor)
  • Make authors bold
  • Test on mobile
  • Fix tables and equations on mobile
  • Make PDF/EPS conversion higher res Make graphics rendering higher resolution #248

Can possibly be deferred to after shipping:

Long term goals:

A rough list of known problems this fixes (!):

Closes #159
Closes #9
Closes #23
Closes #24
Closes #25
Closes #34
Closes #51 (latexml renders tables so, so much better)
Closes #52
Closes #66
Closes #74
Closes #79
Closes #80
Closes #84 (I think? or at least makes it not our problem)
Closes #97
Closes #92
Closes #99
Closes #103
Closes #105
Closes #109
Closes #107
Closes #148
Closes #139
Closes #142
Closes #160
Closes #131
Closes #175
Closes #154

(and probably opens lots of things too)

🎉

@bfirsh bfirsh added this to Backlog in Engrafo via automation Nov 29, 2017
@bfirsh bfirsh moved this from Backlog to In progress in Engrafo Nov 29, 2017
@bfirsh bfirsh force-pushed the latexml branch 2 times, most recently from efb5915 to f3a1e89 Compare December 2, 2017 04:11
This was referenced Dec 7, 2017
@bfirsh
Copy link
Member Author

bfirsh commented Dec 7, 2017

A bit of background on the motivation behind this https://twitter.com/bfirsh/status/938416106687029249?s=17

@andreasjansson
Copy link
Collaborator

belated 👍 👍 👍

@jilljenn
Copy link

jilljenn commented Jun 2, 2018

Thanks for the posterity. I feel like these kinds of links should somehow be featured on the README:

My first question was: why not pandoc? and I somehow got some answers there. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment