feat: Use resize when resampling images with up to 2.56 megapixels#8001
feat: Use resize when resampling images with up to 2.56 megapixels#800172374 wants to merge 1 commit intochatmail:mainfrom
resize when resampling images with up to 2.56 megapixels#8001Conversation
dca6ba7 to
d8397bc
Compare
d8397bc to
ead508e
Compare
|
Just in case, the previous discussion is at #6815 |
|
One question that might be relevant for this PR: If those apply to the image, that could result in downscaling twice; first by the image-editor, to 1536x1536, and then by Core, to 1280x1280, and it would also mean that the change in this PR would apply to all images edited in the Android-app, because the upper limit is 1536x1536. I set this PR back to draft, because #7822 can change what is appropriate to do in this PR. With those changes, images with a resolution of up to 1536×10661 would not be resampled, because that is a lower resolution than 1280x12802. Footnotes |
The
thumbnail-function can resample images quickly, but with very noticeable aliasing when it is used to resample images to a close resolution (~80% of the original resolution, for example). Even if the resolution-difference is large, the quality of the resampling is rather flawed. The function is for quickly resampling lots of images into tiny preview-images, for which the quality is not particularly important, and the resolution-difference is usually large (~20% of the original resolution, for example).The amount of processing (and thus time) required to resample images, depends on the resolution of the original image.
By using the
thumbnail-function only for images with a high resolution, many images will be resampled with the better resampling of theresize-function, while the processing-time, when resampling images with a high resolution, will still be relatively short.