On Wed, Dec 05, 2018 at 08:56:16PM +0900, Akira Yokosawa wrote:
> On 2018/12/05 20:00, Akira Yokosawa wrote:
> > From 213fcdaabb6daa14c1c6cfc6ff780d023478d59f Mon Sep 17 00:00:00 2001
> > From: Akira Yokosawa <akiyks@xxxxxxxxx>
> > Date: Wed, 5 Dec 2018 19:48:32 +0900
> > Subject: [PATCH] count: Include atomic_nehalem.pdf instead of atomic_nehalem.png
> >
> > Without any suffix specified, \includegraphics{} prefers .png to .pdf
> > by default.
> >
> > Fixes: 04e270767c34 ("count: Restore 'fig:count:Atomic Increment Scalability on Nehalem'")
> > Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> > ---
> > Hi Paul,
> >
> > This is a "minor fix". Obviously, I missed to check the restored figure
> > before submitting the previous patch set. ;-)
>
> Looks like this depends on TeX Live version. .png was included on
> Ubuntu Trusty's TeX Live 2013/Debian.
>
> On Ubuntu Bionic (TeX Live 2017/Debian), .pdf is included both with and
> without this fix.
Queued and pushed, thank you!!!
And I must have been blind or something... It shows up in my setup,
but miniaturized. Fixed now, so all is well! ;-)
Thanx, Paul
> Thanks, Akira
>
> >
> > Thanks, Akira
> > --
> > count/count.tex | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/count/count.tex b/count/count.tex
> > index 6fd54b5..bdbc4cd 100644
> > --- a/count/count.tex
> > +++ b/count/count.tex
> > @@ -248,7 +248,7 @@ accuracies far greater than 50\,\% are almost always necessary.
> >
> > \begin{figure}[tb]
> > \centering
> > -\resizebox{2.5in}{!}{\includegraphics{CodeSamples/count/atomic_nehalem}}
> > +\resizebox{2.5in}{!}{\includegraphics{CodeSamples/count/atomic_nehalem.pdf}}
> > \caption{Atomic Increment Scalability on Nehalem}
> > \label{fig:count:Atomic Increment Scalability on Nehalem}
> > \end{figure}
> >
>
