search

dginev / ar5iv

A web service offering HTML5 articles from arXiv.org as converted with latexml
https://ar5iv.org
MIT License
783 stars 20 forks source link

Improve article 2208.14580 #200

Open dginev opened 2 years ago

dginev commented 2 years ago

Exact location of issue

Start of article: "The page layout violates the ICML style". https://ar5iv.labs.arxiv.org/html/2208.14580

Problem details

We are likely interpreting raw a recent ICML style file and emulating PDF-related warnings into the HTML. Likeley needs a small extension to the icml bindings to avoid this.

dginev commented 1 year ago

Another example to handle for this frontmatter issue is 2106.10529 from #265

dginev commented 1 year ago

This seems to be a case of a well-intentioned check that will only be harmless in cases where the actual PDF pipelines are ran and the page dimensions are computed accurately. LaTeXML emulates some of those already, but not all

relevant snippet from mlsys2022.sty:

```tex % Historically many authors tried to include packages like geometry or fullpage, % which change the page layout. It either makes the proceedings inconsistent, or % wastes organizers' time chasing authors. So let's nip these problems in the % bud here. -- Iain Murray 2018. %\RequirePackage{printlen} \AtBeginDocument{% % To get the numbers below, include printlen package above and see lengths like this: %\printlength\oddsidemargin\\ %\printlength\headheight\\ %\printlength\textheight\\ %\printlength\marginparsep\\ %\printlength\footskip\\ %\printlength\hoffset\\ %\printlength\paperwidth\\ %\printlength\topmargin\\ %\printlength\headsep\\ %\printlength\textwidth\\ %\printlength\marginparwidth\\ %\printlength\marginparpush\\ %\printlength\voffset\\ %\printlength\paperheight\\ % \newif\ifmarginsmessedwith \marginsmessedwithfalse \ifdim\oddsidemargin=-16.62178pt \else oddsidemargin has been altered.\\ \marginsmessedwithtrue\fi \ifdim\headheight=10.0pt \else headheight has been altered.\\ \marginsmessedwithtrue\fi \ifdim\textheight=650.43pt \else textheight has been altered.\\ \marginsmessedwithtrue\fi \ifdim\marginparsep=11.0pt \else marginparsep has been altered.\\ \marginsmessedwithtrue\fi \ifdim\footskip=0.0pt \else footskip has been altered.\\ \marginsmessedwithtrue\fi \ifdim\hoffset=0.0pt \else hoffset has been altered.\\ \marginsmessedwithtrue\fi \ifdim\paperwidth=614.295pt \else paperwidth has been altered.\\ \marginsmessedwithtrue\fi \ifdim\topmargin=-24.95781pt \else topmargin has been altered.\\ \marginsmessedwithtrue\fi \ifdim\headsep=10.0pt \else headsep has been altered.\\ \marginsmessedwithtrue\fi \ifdim\textwidth=487.8225pt \else textwidth has been altered.\\ \marginsmessedwithtrue\fi \ifdim\marginparwidth=65.0pt \else marginparwidth has been altered.\\ \marginsmessedwithtrue\fi \ifdim\marginparpush=5.0pt \else marginparpush has been altered.\\ \marginsmessedwithtrue\fi \ifdim\voffset=0.0pt \else voffset has been altered.\\ \marginsmessedwithtrue\fi \ifdim\paperheight=794.96999pt \else paperheight has been altered.\\ \marginsmessedwithtrue\fi \ifmarginsmessedwith \textbf{\large \em The page layout violates the ICML style.} Please do not change the page layout, or include packages like geometry, savetrees, or fullpage, which change it for you. We're not able to reliably undo arbitrary changes to the style. Please remove the offending package(s), or layout-changing commands and try again. \fi} ``` I'll deprioritize until we have more of the native `latex.ltx` raw loading emulated fully.