Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Fix & update yuyv/uyvy encodings #6445

Merged
merged 1 commit into from
Jul 14, 2023
Merged

Fix & update yuyv/uyvy encodings #6445

merged 1 commit into from
Jul 14, 2023

Conversation

jtbandes
Copy link
Member

@jtbandes jtbandes commented Jul 14, 2023

User-Facing Changes

  • Renamed raw image encoding uyuv to uyvy for accuracy.
  • Added support for raw image encoding yuv422_yuy2 as an alias for yuyv.

Description

All YUV 4:2:2 formats encode one U and one V value for every two Y values. The byte orders supported by Studio and by other apps in the ecosystem are [U, Y1, V, Y2] and [Y1, U, Y2, V].

Other references:

@jtbandes jtbandes added the docs Requires documentation update label Jul 14, 2023
Copy link
Contributor

@defunctzombie defunctzombie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we expecting any reports of folks that had recorded data with uyuv? Or is this unlikely as it is not a real format?

@jtbandes
Copy link
Member Author

Seems unlikely to me. uyuv was only added as an alias to the existing (and more common) yuv422, and only 5 months ago.

@jtbandes jtbandes merged commit f2b9900 into main Jul 14, 2023
14 of 15 checks passed
@jtbandes jtbandes deleted the jacob/oy-vey branch July 14, 2023 03:07
jtbandes added a commit to foxglove/schemas that referenced this pull request Jul 14, 2023
Rename `uyuv` to `uyvy` and add `yuv422_yuy2`.

See foxglove/studio#6445
@sgf201
Copy link

sgf201 commented Sep 6, 2023

the YUV422P is useful, @jtbandes ,could you add the 420P and 422p support

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Requires documentation update
Development

Successfully merging this pull request may close these issues.

Is this a mistake? there is no uyuy, does it mean uyvy?
3 participants