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

[manual] add gnuplot files parametric-example-cut to repository? #719

Closed
Mo-Gul opened this issue Jul 25, 2019 · 7 comments
Closed

[manual] add gnuplot files parametric-example-cut to repository? #719

Mo-Gul opened this issue Jul 25, 2019 · 7 comments

Comments

@Mo-Gul
Copy link
Contributor

Mo-Gul commented Jul 25, 2019

Shall the gnuplot files

  • pgf-parametric-example-cut.gnuplot
  • pgf-parametric-example-cut.table

in the folder <doc\generic\pgf\version-for-luatex\en\plots> from

\begin{key}{/tikz/xrange=\meta{start}|:|\meta{end}}
Set the $x$-range. This makes sense only for parametric plots.
%
\begin{codeexample}[]
\tikz \draw[scale=0.5,domain=-3.141:3.141,smooth,xrange=0:1]
plot[parametric,id=parametric-example-cut] function{t*sin(t),t*cos(t)};
\end{codeexample}
%
\end{key}

be added to the repository as well?

@hmenke
Copy link
Member

hmenke commented Jul 27, 2019

Are there any problems if those files are missing? I though the data files were only needed if the plot used the external gnuplot program.

@Mo-Gul
Copy link
Contributor Author

Mo-Gul commented Jul 27, 2019

These files are created by gnuplot. And gnuplot is used in the above example to create the plot. (For details see sections 22.2 and 22.6 in the pgfmanual.)

To answer your question: No, there is no problem if you have gnuplot installed and working. But if not, then the .table file can used instead using \tikz \draw plot [...] file {<file name>.table};.

In this case I see two options to be consistent (again): Either add the above files, too, or delete all the others.

Another pro point is that these files are some kind of a test. Thus the .gnuplot files are tests that TikZ/PGF still does what it should and the .table files are tests that gnuplot still does what it should.

@hmenke
Copy link
Member

hmenke commented Jul 30, 2019

Okay, I will add them then. I also noticed that some gnuplot files are overwritten when building the documentation. This might have to do with the gnuplot version change from 4.x to 5.x some time in the last years.

@Mo-Gul
Copy link
Contributor Author

Mo-Gul commented Jul 30, 2019

Thanks.

The gnuplot files should be overwritten each time you compile, i.e. when you compile with LuaLaTeX the files in doc/generic/pgf/version-for-luatex/en/plots will all be recreated. Of course the same holds for the other engines.

Thus, I don't know if/when the gnuplot files under doc/generic/pgf/text-en/plots are created/used. Maybe this is a leftover when it was allowed to right in higher folders than the main file? This would be a perfect explanation, because all the plot files in all the different folders are identical ...

@hmenke hmenke added this to the 3.1.5 milestone Nov 3, 2019
hmenke added a commit to hmenke/pgf that referenced this issue Nov 4, 2019
@hmenke hmenke closed this as completed Nov 4, 2019
@muzimuzhi
Copy link
Member

Commit f7a24c5 only added file pgf-parametric-example-cut.table under path for driver dvipdfmx. See https://github.com/pgf-tikz/pgf/blob/master/doc/generic/pgf/version-for-dvipdfmx/en/plots/pgf-parametric-example-cut.table. For other drivers, especially luatex, this file is still missing.

Therefore texlua build.lua manual luatex will write following warning to log,

Package pgf Warning: Plot data file `plots/pgf-parametric-example-cut.table' not found. on input line 1.

and cause the corresponding drawing missing from the output PDF (pgfmanual v3.1.6a, page 348).
image

This can be fixed either by copying/linking that file to all drivers, or installing gnuplot and enabling shell-escape in CI.

@Mo-Gul
Copy link
Contributor Author

Mo-Gul commented Nov 26, 2020

Unfortunately a fix to this issue didn't make it into v3.1.7 ...

@muzimuzhi, do you have a clue how to extend the build script to also "install" gnuplot so the .gnuplot and .table files are generated on the fly? (This should be the long-term solution so also changes in gnuplot will be detected when the test-suite is working.)

If also Henri doesn't have a clue how to do this or doesn't find the time to implement it "soon" the temporary fix should be to (re)add the .table file(s).

@muzimuzhi
Copy link
Member

Fixed in eee4a6a which is included from 3.1.7a, so I will close this.

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

No branches or pull requests

3 participants