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

Eps files #2

Open
orybkin opened this issue Feb 12, 2019 · 6 comments
Open

Eps files #2

orybkin opened this issue Feb 12, 2019 · 6 comments
Labels
enhancement New feature or request

Comments

@orybkin
Copy link

orybkin commented Feb 12, 2019

Hi,

Thanks for open-sourcing this! I have a feature request related to .eps files. My understanding that the common installations of TeX convert each existing myfile.eps into a file myfile-eps-converted-to.pdf, and this file is then looked up when the pdf needs to be compiled.

When I try to use the arxiv-latex-cleaner on a LaTeX project with .eps files, the *-eps-converted-to.pdf are generated in the old project folder, but not the new cleaned-up folder. I am also not sure whether these files are compressed.

The minimal fix I am looking for is to automatically move the *-eps-converted-to.pdf files from the old to the new folder. I might eventually fix this myself, but just letting you know about this problem. Thanks!

@jponttuset
Copy link
Collaborator

Hi Oleh, thanks for commenting!
What's the usual way to work with eps files (sorry, I haven't used them in a while)?
You include the figure with \includegraphics without extension, then pdflatex converts them automatically, and finally the \includegraphics is clever enough to include the *-eps-converted-to.pdf?

In this case, what would need to be done is to check the eps files that are referenced and copy the matching converted PDF.

@jponttuset jponttuset added the enhancement New feature or request label Feb 23, 2019
@krono
Copy link

krono commented Feb 23, 2019

The cleaner could copy the *-eps-converted-to.pdf alongside the *.eps, but as *.pdf (if the latter not exists already)

@orybkin
Copy link
Author

orybkin commented Mar 24, 2019

Jordi,

Sorry for not answering earlier, my notification settings were off.

I am not an expert on LaTeX either, but I think what you're describing is correct.

Tobias,

I think this won't work because LaTeX compilation (not sure which function though) expects the *-eps-converted-to.pdf file. I think what is needed is simply to copy the *-eps-converted-to.pdf, but I might be wrong

@jponttuset
Copy link
Collaborator

Hi Oleh,
Might give it a shot at some point. For now, I'll leave this open as enhancement in case someone would like to chip in.

@krono
Copy link

krono commented Mar 24, 2019

@orybkin
It depends:
If you do

\includegraphics{foo}

then pdflatex will prefer foo.pdf and latex will prefer foo.eps. (not sure about XeTeX, tho)

sebymiano added a commit to sebymiano/arxiv-latex-cleaner that referenced this issue Jun 18, 2019
This commit introduces the possibility to execute commands_to_delete
with nested commands (issue google-research#9) by improving the regex expression
that matches the user-defined custom command.
It also improves the regex expression used to match comments environment
that was affected by the same problem.

Limitation google-research#1: in commands_to_delete max 2 nested levels are supported.
Limitation google-research#2: the \end{comment} has to be followed by a newline character.
sebymiano added a commit to sebymiano/arxiv-latex-cleaner that referenced this issue Jun 18, 2019
This commit introduces the possibility to execute commands_to_delete
with nested commands (issue google-research#9) by improving the regex expression
that matches the user-defined custom command.
It also improves the regex expression used to match comments environment
that was affected by the same problem.

Limitation google-research#1: in commands_to_delete max 2 nested levels are supported.
Limitation google-research#2: the \end{comment} has to be followed by a newline character.
@peymmo
Copy link

peymmo commented Jul 29, 2020

I am echoing also this issue. *.eps files are tricky and depends on which latex compiler you use. Better to pass the eps files as it is like another image format I guess and don't delete them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants