Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-115723

VideoOutput.orientation seems to report an incoherent value

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P3: Somewhat important P3: Somewhat important
    • 6.8
    • 6.5.2
    • Multimedia
    • None
    • Android
    • Multimedia, weeks 18-19

      VideoOutput.orientation reports a value that seems not to be coherent with frames provided by the QVideoSink.

      My understanding is that a change from Qt 6.5.1 was made so that, on Android, the camera stream is reported as landscape instead of portrait (when the device is in portrait orientation), and the information about the needed transformation is carried inside the frame. While each QVideoFrame includes a reasonable orientation, VideoOutput remains 0.

      With this minimal example https://github.com/carlonluca/CameraTest I get these logs with Qt 6.5.1:

       08-02 13:08:42.149  D  [5893/5924] qml      : Video stream: QRectF(0, 0, 1088, 1920)
       08-02 13:08:42.149  D  [5893/5924] qml      : Video orientation: 0
       08-02 13:08:42.205  D  [5893/5924] libappCameraTest_arm64-v8a.so Frame QSize(1088, 1920) 0
       08-02 13:08:42.219  I  [5893/5924] libappCameraTest_arm64-v8a.so Got image from video frame of size: QSize(1088, 1920)

      and these from Qt 6.5.2:

       08-02 13:09:47.287  D  [6517/6533] qml      : Video stream: QRectF(0, 0, 1920, 1088)
       08-02 13:09:47.287  D  [6517/6533] qml      : Video orientation: 0
       08-02 13:09:47.300  D  [6517/6533] libappCameraTest_arm64-v8a.so Frame QSize(1920, 1088) 90
       08-02 13:09:47.320  I  [6517/6533] libappCameraTest_arm64-v8a.so Got image from video frame of size: QSize(1088, 1920)

      The change from 6.5.1 may be reasonable, but I'd expect VideoOutput.orientation to be 90 as well in Qt 6.5.2.

      Feel free to close immediately if my understanding is incorrect.

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            bartlomiejmoskal Bartlomiej Moskal
            luc4 Luca Carlon
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes