Skip to content

BUG: Fix error in cmap extraction#1544

Merged
MartinThoma merged 5 commits intopy-pdf:mainfrom
pubpub-zz:indexerr_cmap_1533
Jan 21, 2023
Merged

BUG: Fix error in cmap extraction#1544
MartinThoma merged 5 commits intopy-pdf:mainfrom
pubpub-zz:indexerr_cmap_1533

Conversation

@pubpub-zz
Copy link
Copy Markdown
Collaborator

fixes #1533 and late #1091

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 9, 2023

Codecov Report

Base: 91.79% // Head: 91.79% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (753514d) compared to base (e7e4ffc).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1544   +/-   ##
=======================================
  Coverage   91.79%   91.79%           
=======================================
  Files          33       33           
  Lines        6093     6094    +1     
  Branches     1200     1200           
=======================================
+ Hits         5593     5594    +1     
  Misses        323      323           
  Partials      177      177           
Impacted Files Coverage Δ
pypdf/_cmap.py 95.25% <100.00%> (+0.02%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pubpub-zz
Copy link
Copy Markdown
Collaborator Author

@MartinThoma ,
just waiting for feeback on #1533 before merging

@MartinThoma
Copy link
Copy Markdown
Member

The feedback was:

I've checked with all of my invoice types and works well. Thanks for the correction!

So we are good to go :-)

@MartinThoma MartinThoma changed the title fix error in cmap extraction BUG: Fix error in cmap extraction Jan 10, 2023
@pubpub-zz
Copy link
Copy Markdown
Collaborator Author

@MartinThoma
this PR is ready for merge, candidate for next release

@MartinThoma
Copy link
Copy Markdown
Member

You're on a roll! Thanks again 🤗

@MartinThoma MartinThoma merged commit c1f8742 into py-pdf:main Jan 21, 2023
@MartinThoma MartinThoma added the is-bug From a users perspective, this is a bug - a violation of the expected behavior with a compliant PDF label Jan 21, 2023
MartinThoma added a commit that referenced this pull request Jan 22, 2023
New Features (ENH):
-  Add page label support to PdfWriter (#1558)
-  Accept inline images with space before EI (#1552)
-  Add circle annotation support (#1556)
-  Add polygon annotation support (#1557)
-  Make merging pages produce a deterministic PDF (#1542, #1543)

Bug Fixes (BUG):
-  Fix error in cmap extraction (#1544)
-  Remove erroneous assertion check (#1564)
-  Fix dictionary access of optional page label keys (#1562)

Robustness (ROB):
-  Set ignore_eof=True for read_until_regex (#1521)

Documentation (DOC):
-  Paper size (#1550)

Developer Experience (DEV):
-  Fix broken combination of dependencies of docs.txt
-  Annotate tests appropriately (#1551)

[Full Changelog](3.2.1...3.3.0)
@pubpub-zz pubpub-zz deleted the indexerr_cmap_1533 branch June 24, 2023 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is-bug From a users perspective, this is a bug - a violation of the expected behavior with a compliant PDF

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PyPDF2 throws exception during extract_text()

2 participants