Skip to content

Output as PDF annotations #50

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

Open
wants to merge 2 commits into
base: primary
Choose a base branch
from

Conversation

benoitrosa
Copy link

Hi,

Atfer posting an issue on the original repo, I went ahead and actually implemented the feature, which proved quite easy. This pull request is the basic functionality. I tested both on a very simple Latex MWE with just a few words, and with a very complex scientific publication which was changed from top to bottom between two revisions, and the output is clean :)

Main change made: a new CLI option -p (or --pdfoutput). If activated, it will output two PDFs in the working directory named output1.pdf and output2.pdf. The output PDFs are a copy of the input ones with changes highlighted as annotations.

Dependency on the pdf-annotate python library is necessary (https://github.com/plangrid/pdf-annotate), which in turn requires Python3.8 (it's a pure python library). As of now the import of the library is done in the beginning, perhaps it could be done only if the -p option is used, so that pdf-diff can still be used with PNG output without requiring the lib.

Tests done on Ubuntu 20.04 with python 3.8

@JoshData
Copy link
Owner

JoshData commented Jul 5, 2023

This is very cool but since I'm not actively working on this project it'll be a long time before I take a close enough look to consider merging it. But thanks!

@benoitrosa
Copy link
Author

No problem, I needed the feature anyway so it's my pleasure ! Perhaps I'll implement a few more things and complete this PR if I find some time, if it can be useful for other people ;)

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

Successfully merging this pull request may close these issues.

2 participants