Skip to content
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

Fix cropping when exporting #16689

Closed
redmoon1945 opened this issue Apr 26, 2024 · 3 comments · Fixed by #16748
Closed

Fix cropping when exporting #16689

redmoon1945 opened this issue Apr 26, 2024 · 3 comments · Fixed by #16748
Labels
reproduce: confirmed a way to make the bug re-appear 99% of times has been found

Comments

@redmoon1945
Copy link

redmoon1945 commented Apr 26, 2024

The "Set size in pixels" in Export does not always produce the expected results. If we set width and height, sometimes none of the specified values will be matched. For example, I have a NEF image of size 8288x5520. If I try to export with “set size in pixel” = 1860x1020, I get an image of size 1531x1019, when the correct result should be 1531x1020. This is annoying when for example you want to export images for HDTV using 100% of the real estate space (meaning either the width or the height must be 100%). Running Flatpak 4.6.1 on Linux Mint 21.3.
ImageMagick gives the proper result :
convert image.tif -resize 1860x1020 a.jpg
the final dimension is 1531 x 1020.

I suggest that this bug be fixed (inspecting ImageMagick resize code could help here). An alternative avenue would be that we add an option in the export Menu, kind of a checkbox "Stick to at least one side" if the bug cannot be fixed.

See discussion : https://discuss.pixls.us/t/bug-with-set-size-in-pixel-for-export/43283/1

@jenshannoschwalm
Copy link
Collaborator

  1. There are already some reports here about export size and cropping, are you sure this is different?
  2. The tooltips tell for the size in pixels something like "maximum limit", if you just define the height as 1020 and leave the width at 0 should give exported file always the defined height. Does this give inconsistant sizes?

@redmoon1945
Copy link
Author

redmoon1945 commented Apr 27, 2024

I have tried to leave the width to 1020, but no change, the result is still 1019. I can provide the .NEF if you want, no problem. From what I can see searching the issue, this is not a duplicate. Thx

@jenshannoschwalm
Copy link
Collaborator

Can reproduce, depends on provided size - either by crop iop or provided data from raw or after rawprepare tuning.

  1. not sure how to fix ...
  2. basically a duplicate of Image export size doesn't respect crop aspect ratio #16638, it's all the same story and we have had more reports before.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reproduce: confirmed a way to make the bug re-appear 99% of times has been found
Projects
None yet
2 participants