Fix default orientation and box white balance

[remia]

Description

Proposing the follow minor changes:

• Change the --flip default value from 0 to -1, meaning it will follow the metadata instead of disabling the flip (0 means none), see https://github.com/LibRaw/LibRaw/blob/master/doc/API-datastruct.html#L740
• Pass use_auto_wb = 1 when using box white balancing method, see https://github.com/AcademySoftwareFoundation/OpenImageIO/blob/main/src/raw.imageio/rawinput.cpp#L527

Tests

Checklist:

• I have read the contribution guidelines.
• I have updated the documentation, if applicable. (Check if there is no
  need to update the documentation, for example if this is a bug fix that
  doesn't change the API.)
• I have ensured that the change is tested somewhere in the testsuite
  (adding new test cases if necessary).
• My code follows the prevailing code style of this project. If I haven't
  already run clang-format before submitting, I definitely will look at the CI
  test that runs clang-format and fix anything that it highlights as being
  nonconforming.

[linux-foundation-easycla]

• ❌ - login: @remia / name: Rémi Achard . The commit (45d4a8b, f89133c) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.77%. Comparing base (4202095) to head (45d4a8b).

Files with missing lines	Patch %	Lines
src/rawtoaces_util/image_converter.cpp	50.00%	2 Missing ⚠️

❌ Your patch status has failed because the patch coverage (60.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #268   +/-   ##
=======================================
  Coverage   90.77%   90.77%           
=======================================
  Files          17       17           
  Lines        3446     3446           
  Branches      532      532           
=======================================
  Hits         3128     3128           
  Misses        318      318           

Files with missing lines	Coverage Δ
include/rawtoaces/image_converter.h	100.00% <100.00%> (ø)
src/rawtoaces_util/image_converter.cpp	78.44% <50.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4202095...45d4a8b. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[soswow]

Is there a way we can have a regression tests for changed functionality? Also, this feels like a breaking API move basically - changing default behaviour.

[remia]

I'm not sure there is an easy to test this but will have a look at the test suite.

Also yes this changes behaviour, considering these are bug fix so hopefully this is for the better (the greybox white balance was non functional and the orientation from multiple conversion I tested was wrong, though if some are relying on incorrect orientation metadata in the source file, the latter change might be an issue but for the general case I'd think it's something we want).

[antond-weta]

I'm afraid there is no easy way to test orientations. We don't have any vertical images in the test suite, and ideally we would need multiple. Implementing generation of fake raw test images is on the road map, but until then there is not much we can do.

[remia]

Ok, I'll leave this here for the time being then, trying to sort out the CLA thing.