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

[Bug]: Video is flipped vertically when playing #938

Closed
david-swift opened this issue May 5, 2024 · 6 comments · Fixed by #1308
Closed

[Bug]: Video is flipped vertically when playing #938

david-swift opened this issue May 5, 2024 · 6 comments · Fixed by #1308
Labels
bug Something isn't working

Comments

@david-swift
Copy link
Contributor

Describe the bug

When playing the video in this post, it is flipped vertically in the player view. This doesn't happen with other videos and images.

Steps To Reproduce

  1. Search for the post https://mastodon.gamedev.place/@jerrodputman/112381197588667762
  2. Click on the video thumbnail to play the video

Logs and/or Screenshots

Screencast.from.2024-05-05.18-26-20.webm

There's no output when playing the video. This is printed when opening the app (probably not helpful):

(dev.geopjr.Tuba:2): Tuba-CRITICAL **: 18:36:14.225: Network.vala:77: Request "https://mastodon.online/api/v1/markers?timeline[]=notifications" failed: 403 Forbidden

(dev.geopjr.Tuba:2): Gtk-CRITICAL **: 18:36:14.575: gtk_list_box_remove: assertion 'GTK_IS_WIDGET (child)' failed

(dev.geopjr.Tuba:2): Tuba-CRITICAL **: 18:36:14.588: tuba_views_content_base_real_on_create_model_widget: assertion 'obj != NULL' failed

(dev.geopjr.Tuba:2): GLib-GObject-CRITICAL **: 18:36:14.588: g_object_is_floating: assertion 'G_IS_OBJECT (object)' failed

(dev.geopjr.Tuba:2): Gtk-CRITICAL **: 18:36:14.588: gtk_widget_set_visible: assertion 'GTK_IS_WIDGET (widget)' failed

(dev.geopjr.Tuba:2): Gtk-CRITICAL **: 18:36:14.588: gtk_list_box_insert: assertion 'GTK_IS_WIDGET (child)' failed

(dev.geopjr.Tuba:2): GLib-GObject-CRITICAL **: 18:36:14.588: g_object_unref: assertion 'G_IS_OBJECT (object)' failed

(dev.geopjr.Tuba:2): GLib-GObject-CRITICAL **: 18:36:14.588: g_object_unref: assertion 'G_IS_OBJECT (object)' failed

(dev.geopjr.Tuba:2): Gtk-WARNING **: 18:36:23.767: GtkImage 0x563dd59af020 (image) reported min height -17, but sizes must be >= 0

Instance Backend

Mastodon

Operating System

Vanilla OS 2.0

Package

Flatpak

Troubleshooting information

os: GNOME 46 (Flatpak runtime)
prefix: /app
flatpak: true
version: 0.7.2 (production)
gtk: 4.14.3 (4.14.1)
libadwaita: 1.5.0 (1.5.0)
libsoup: 3.4.4 (3.4.4)
libgtksourceview: 5.12.0 (5.12.0)

Additional Context

No response

@david-swift david-swift added the bug Something isn't working label May 5, 2024
@GeopJr
Copy link
Owner

GeopJr commented May 5, 2024

Looking at the exif data, it reports itself as upside down:
Rotation: 180

Assuming that was shot on an apple device, it wouldn't be the first time they do this. All media shot on apple devices seem to be upside down and change their orientation using exif metadata. For images, we do actually correct them with Gdk.Pixbuf#apply_embedded_orientation

Now for video, I don't know 🤷

@rmader
Copy link
Contributor

rmader commented Jun 25, 2024

Now for video, I don't know 🤷

Gstreamer uses the image-orientation tag for such cases. There are multiple ways to apply the rotation - such as using the glvideoflip element or having direct support in the sink (see this MR for the gtk4paintablesink).

Unfortunately neither the GTK internal player nor Clapper seem to support it yet (which is an issue with videos from other devices aswell).

@GeopJr
Copy link
Owner

GeopJr commented Jun 25, 2024

Thanks for the info!

Rafostar gave me a lengthy explanation on orientation when this issue opened on Matrix, I'm sure they'll be interested in this, @Rafostar

@Rafostar
Copy link
Contributor

nor Clapper seem to support it

This is incorrect. Clapper does support this. In fact this issue is not present if you build Tuba with Clapper support (its set behind meson option).

Obligatory screenshot 😄 :
image

@rmader
Copy link
Contributor

rmader commented Jun 25, 2024

This is incorrect. Clapper does support this.

Oh, then it's just the version currently shipped by fedora here :/ Great to hear that it works!

@GeopJr
Copy link
Owner

GeopJr commented Feb 5, 2025

fixed by: #1308

@GeopJr GeopJr closed this as completed Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants