jpg画像の読み込み時に回転値を反映してほしい

[agehama]

予期しない動作の内容 | Describe the unexpected behavior
jpgのExifに含まれる回転値（Orientation）が考慮されないため、この値が設定された画像ではブラウザと異なる表示結果になります

再現できる画像：

表示の比較：

[Raclamusi]

Discord でこの機能について話したことがあり、回転量取得のコストの検証が必要だという話になりました（が、放置してます。すみません）。

https://discord.com/channels/443310697397354506/998714158621147237/1290958726076825673

[Raclamusi]

雑な実装ですが、既存の Exif 全体を読む機能を使った実装と回転量だけ読む実装で実行速度を比較しました。
ファイルサイズによるでしょうが、かなり差が出そうです。

https://gist.github.com/Raclamusi/1c37a3e0902541be428682fffd475568

--- start
Exif::Read(FilePathView): 3.903 ms
Exif::Read(const IReader&): 2.732 ms
ReadExifOrientation(FilePathView): 0.211 ms
ReadExifOrientation(IReader&): 0.002 ms
--- finish

[Raclamusi]

使用した画像
https://drive.google.com/drive/folders/1MlzWMoQ6eovOIFd-nBATMirytZ_VFBt_?usp=drive_link

[Reputeless]

調査ありがとうございます。Exif クラスの使用はかなり非効率ですね。
JPEGDecoder.cpp 内または public な API に回転取得専用の関数を用意したいところです。

• ① 既存の JPEG デコーダの実装方式（ストリーミング）と相性が悪そう
• ② getImageInfo() でもやるのか

といった課題が残ります。
https://github.com/Siv3D/OpenSiv3D/blob/main/Siv3D/src/Siv3D/ImageFormat/JPEG/JPEGDecoder.cpp

[Reputeless]

Siv3D の JPEG デコーダはファイルの内容を全部読み込んでいましたね。別のデコーダと混同していました。① は取り消します。
② についても、回転情報がファイルの冒頭にあるならそこまで問題にならないと思います。

[Reputeless]

ReadExifOrientation(IReader&) を JPEGDecoder.cpp に実装し、
decode では MemoryViewReader を渡して、getImageInfo では IReader を渡して使うのが良いのかなと思いました。
decode では、回転情報に応じて Image を加工して返すようにしましょう。

[Raclamusi]

Image の加工のコストってどうなんでしょう？
Texture にしてから回転、反転した方が速い気がするので、 decode で Image が加工されない選択肢があってもいいと思います。

[Reputeless]

加工にあたっては、若干のコストは避けられないと思います。
しかし Texture を EXIF 情報に合わせて回転・反転させるようなコードをわざわざ書きたいユーザはそんなにいないと思うので、加工済みオンリーで良さそうな気がします。

[Reputeless]

今回の Issue は、Siv3D が Web ブラウザやビューアとは異なる画像を出したことでユーザを驚かせたことが本質なので、そこの解決にフォーカスしたいと思います。
加工しない生の画像を取得するオプションの可能性については、結論は出しません。

JPEG の回転対応は、予期せぬ影響を受けるユーザがいるかもしれないため、v0.8 から対応したいです。
ReadExifOrientation() については Raclamusi さんのコードなので、v8 のほうにプルリクいただくか、あるいはここで提供を表明していただければ移植します。

[Raclamusi]

了解です。
siv8 にプルリクしようと思います。

[Raclamusi]

実装を改良しました。
https://gist.github.com/Raclamusi/1c37a3e0902541be428682fffd475568/494a9831c3cbe23bdce84913c220729310a5c43c

siv8 へのプルリクは、 Image を反転、回転などするメンバ関数が未実装であるため、断念しました。

ReadExifOrientation() のコードを Siv3D の実装のために提供しますので、 Image のメンバ関数を実装されるタイミングで移植していただければと思います。
https://gist.github.com/Raclamusi/1c37a3e0902541be428682fffd475568

[Reputeless]

確かに未実装でした！ 実装時に移植します。