dginev / ar5iv

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

Improve article 1909.11652 (mdframed.sty) #266

Open yuxi-liu-wired opened 2 years ago

yuxi-liu-wired commented 2 years ago

Exact location of issue Multiple figures are wrong.

Problem details Figure 2, 3 contains unrendered subfigures.

Figure 5. caption is duplicated. Figure 7 caption is duplicated. Figure 5,6,7,8 are mashed together incorrectly. They should look like image

Desktop (please complete the following information)

dginev commented 2 years ago

Thanks for the report!

The paper also needs mdframed.sty support for latexml, and indeed - multiple image post-processing errors are reported in the conversion log.

dginev commented 1 year ago

Taking Figures 5 and 6 as an example, they are realized as a singe {figure} consisting of two side-by-side {minipage} panels - but each panel is captioned and numbered as if it were a standalone Figure. We need to extend our flex figure model in latexml to also cover this case:

\begin{figure}
\centering
% \vspace{-0.5in}
\begin{minipage}{.3\textwidth}
\includegraphics[height=3.4cm]{figs/plots/dclaw_randStart.pdf}
\vspace{-5mm}
\caption{\footnotesize{Most methods learn this easier task of valve turning, though our method learns fastest.}}
\label{fig:valve}
\end{minipage} \hspace{0.1in}
\begin{minipage}{.65\textwidth}
\includegraphics[height=3.4cm]{figs/plots/cube_2angles.pdf}
\includegraphics[height=3.4cm]{figs/plots/cube_8angles.pdf}
\vspace{-2mm}
\caption{\footnotesize{In-hand reorientation of a cube. Our method achieves the best results with 2 and 8 goal angles, and model-free algorithms and methods that directly learn a policy, such as MBPO, struggle with 8 goal angles.}}
\label{fig:cube}
\end{minipage}
% \vspace{-3mm}
\end{figure}
dginev commented 7 months ago

The figure panels are improved after the latest ar5iv update, but not enough to fully match the PDF layout.

In addition, our reduced image scaling options have made some of the assets too grainy (could be good tests). Leaving open for now.