Skip to content

Conversation

@CyMule
Copy link
Contributor

@CyMule CyMule commented Jan 7, 2026

Fixes ~15-18% performance regression introduced in 20251230 where f-strings were evaluated eagerly even when logging was disabled.

See: pdfminer/pdfminer.six#1233
Fix: pdfminer/pdfminer.six#1234


Note

Addresses a pdf parsing performance issue by updating dependency and aligning versioning.

  • Upgrade pdfminer-six to 20260107 in requirements/base.txt and constrain it in requirements/constraints.in (perf fix)
  • Bump version to 0.0.92 in prepline_general/api/__version__.py and preprocessing-pipeline-family.yaml
  • Update CHANGELOG.md with the perf fix note

Written by Cursor Bugbot for commit cd8a7ef. This will update automatically on new commits. Configure here.

Fixes ~15-18% performance regression introduced in 20251230 where
f-strings were evaluated eagerly even when logging was disabled.

See: pdfminer/pdfminer.six#1233
Fix: pdfminer/pdfminer.six#1234
@socket-security
Copy link

socket-security bot commented Jan 7, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedpypi/​pdfminer-six@​20251230 ⏵ 2026010798 +1100100100100

View full report

@CyMule CyMule merged commit e464c16 into main Jan 7, 2026
12 checks passed
@CyMule CyMule deleted the perf/upgrade-pdfminer-six-20260107 branch January 7, 2026 16:41
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.

3 participants