Skip to content

Fix 2-up printing#78

Merged
Lucas-C merged 5 commits intopy-pdf:mainfrom
pastor-robert:up2fix
Dec 3, 2024
Merged

Fix 2-up printing#78
Lucas-C merged 5 commits intopy-pdf:mainfrom
pastor-robert:up2fix

Conversation

@pastor-robert
Copy link
Copy Markdown
Contributor

Fixed a call to merge_translated_page. Apparently, the switch from PyPDF2 to pypdf changed the signature of merge_translated_page, adding the over parameter. Change the call to use keyword parameters.

The symptom was 2-up only showed one page per sheet, on the left side, with blank space on the right.

@Lucas-C
Copy link
Copy Markdown
Member

Lucas-C commented Dec 2, 2024

Hi @pastor-robert

Thank you very much for your contribution 👍

Could you maybe use this opportunity to add a unit test, by initatiating a tests/test_up2.py inspired by the other existing tests in the tests/ directory?

Also, the GitHub Actions pipeline is failing, but just running black pdfly/pdfly/up2.py should solve it.

@pastor-robert
Copy link
Copy Markdown
Contributor Author

Could you maybe use this opportunity to add a unit test, by initatiating a tests/test_up2.py inspired by the other existing tests in the tests/ directory?

Sure. I'll do that tonight.

Also, the GitHub Actions pipeline is failing, but just running black pdfly/pdfly/up2.py should solve it.

Thanks. I'll run black.

@Lucas-C Lucas-C merged commit c05a660 into py-pdf:main Dec 3, 2024
@Lucas-C
Copy link
Copy Markdown
Member

Lucas-C commented Dec 3, 2024

Good job on this 👍

Thank you very much for your contribution!

@pastor-robert pastor-robert deleted the up2fix branch December 4, 2024 05:54
MartinThoma added a commit that referenced this pull request Dec 8, 2024
## What's new

### New Features (ENH)
- New `booklet` command to adjust offsets and lengths ([PR #77](#77))
- New `uncompress` command ([PR #75](#75))
- New `update-offsets` command to adjust offsets and lengths ([PR #15](#15))
- New `rm` command ([PR #59](#59))
- `metadata`: now also displaying CreationDate, Creator, Keywords & Subject ([PR #73](#73))
- Add warning for out-of-bounds page range in pdfly `cat` command ([PR #58](#58))

### Bug Fixes (BUG)
- `2-up` command, that only showed one page per sheet, on the left side, with blank space on the right ([PR #78](#78))

[Full Changelog](0.3.3...0.4.0)
@Lucas-C
Copy link
Copy Markdown
Member

Lucas-C commented Dec 8, 2024

This has been released in version 0.4.0: https://pypi.org/project/pdfly/#history

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

Labels

None yet

2 participants