Skip to content

WIP:Pdf optimizations#780

Merged
aaronleopold merged 5 commits intostumpapp:breaking/sea-ormfrom
hollisticated-horse:pdf-optimizations
Oct 13, 2025
Merged

WIP:Pdf optimizations#780
aaronleopold merged 5 commits intostumpapp:breaking/sea-ormfrom
hollisticated-horse:pdf-optimizations

Conversation

@hollisticated-horse
Copy link
Copy Markdown
Contributor

This is a new PR, doubling down on pdfium as backend. It provides, I hope, some improvements to PDF rendering.

@aaronleopold
Copy link
Copy Markdown
Collaborator

Sorry I haven't gotten to looking at this yet, I'm hoping to have some time tomorrow to give it a review

Copy link
Copy Markdown
Collaborator

@aaronleopold aaronleopold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall I think this looks good, I had a couple of suggestions but nothing major. I'll try and actually test this out and see how it renders things during the week.

Also if you prefer I try and make the suggested changes let me know, I don't mind

@hollisticated-horse
Copy link
Copy Markdown
Contributor Author

hollisticated-horse commented Oct 8, 2025

running the build process to check everythings works. I did things the wrong way around.

EDIT: The build compiles and works.

The page rendering performances are still not great imo. I havn't found the perfect settings yet. But it's much better then before.

- Add configurable DPI, dimensions, and output format settings
- Implement disk caching system with atomic file operations
- Add pre-rendering of adjacent pages for faster navigation
- Support multiple quality modes (high quality vs fast rendering)
- Add smart cache key generation based on file metadata
- Implement background pre-rendering tasks with concurrency limits
- Support WebP, PNG, and JPEG output formats
- Add environment variable controls for all PDF rendering options
- Route PDF files through enhanced caching system for better performance

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@hollisticated-horse hollisticated-horse changed the title Pdf optimizations WIP:Pdf optimizations Oct 8, 2025
Copy link
Copy Markdown
Collaborator

@aaronleopold aaronleopold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! I'll do a little local testing before merging it in, thanks!

@aaronleopold aaronleopold merged commit e6f7d63 into stumpapp:breaking/sea-orm Oct 13, 2025
2 checks passed
@hollisticated-horse hollisticated-horse deleted the pdf-optimizations branch October 16, 2025 16:16
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