con / opfvta-reexecution

Container-based Replication of https://doi.org/10.1038/s41398-022-01812-5
Apache License 2.0
1 stars 1 forks source link

We need to include `graphviz` in the container for dotfile compilation. #8

Closed TheChymera closed 1 year ago

TheChymera commented 1 year ago

We now have different targets in the Makefile, for building the article on the system, or inside the container: make article and make container-article. The former works if graphviz is present on the system, the latter universally fails right now.

Any ideas how best to include graphviz in the image @asmacdo ? It's this bad boy here → https://gitlab.com/graphviz/graphviz/

asmacdo commented 1 year ago

Here's where we build the latex image.

https://github.com/con/opfvta-replication-2023/blob/master/code/images/Containerfile.latex

TheChymera commented 1 year ago

As of https://github.com/con/opfvta-replication-2023/commit/afaeaeddc9240af815086853d3d0f9016fb8f421 , this still fails:

Latexmk: 'pdflatex': source 'data/topology' doesn't exist.
   I'll try making it with allowed extensions
------------
Running 'make "data/topology.jpg"'
------------
make: *** No rule to make target 'data/topology.jpg'.  Stop.
------------
Running 'make "data/topology.pdf"'
------------
pushd data; dot -Tpdf topology.dot -o topology.pdf 2> topology.log
/data/data /data
Makefile:41: recipe for target 'data/topology.pdf' failed
make: *** [data/topology.pdf] Error 127
------------
Running 'make "data/topology.png"'
------------
make: *** No rule to make target 'data/topology.png'.  Stop.
------------
Running 'make "data/topology.tex"'
------------
make: *** No rule to make target 'data/topology.tex'.  Stop.
Latexmk: Summary of warnings:
  Latex failed to resolve 15 citation(s)
Latexmk: Errors, in force_mode: so I tried finishing targets
Collected error summary (may duplicate other messages):
  biber article: Biber errors: See file 'article.blg'
  pdflatex: Command for 'pdflatex' gave return code 256
make: *** [Makefile:44: container-article] Error 12
[deco]~/docsrc/opfvta-replication-2023/paper/source ❱ cat data/topology.log
/bin/bash: dot: command not found

It appears to be the correct package name → https://manpages.debian.org/stretch/graphviz/dot.1.en.html

asmacdo commented 1 year ago

Completed