-
Notifications
You must be signed in to change notification settings - Fork 102
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
[Up-477] Optimization example - remove unused resources #229
[Up-477] Optimization example - remove unused resources #229
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update the README also under compress/ folder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also would make sense to enable this flag on:
https://github.com/unidoc/unipdf-examples/blob/master/compress/pdf_optimize.go
Maybe we should mention which filters are lossy vs lossless. Removing unused resources is lossless for the most parts. |
@kellemNegasi can you update this and the other PR? A new release of unipdf has just been published so this should be ready once updated. |
Ok. I will do. |
@gunnsth I have tried to mention which filters are lossless on the README file. Can you check and see if I have to add or remove some filters to or from the list. |
compress/README.md
Outdated
} | ||
``` | ||
|
||
From the available filters `CleanFonts`, `CleanUnusedResources`, `CombineDuplicateStreams`, `CombineDuplicateDirectObjects`,`CombineDuplicateDirectObjects`, `CleanFonts` and `CompressStreams` enable lossless compressions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UseObjectStreams also
CombineIdenticalIndirectObjects also
Actually most of them are lossless except ImageUpperPPI and ImageQuality.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.