Bug#985358: pitivi: A/V out of sync in rendered videos
On Sat, Mar 20, 2021 at 10:43:05AM +0200, Sebastian Dröge wrote: > On Tue, 2021-03-16 at 13:39 -0300, Antonio Terceiro wrote: > > As a user, I don't even know about all the stuff that could be better; > pitivi worked ok for me as an amateur video editor. It just failed at > the end of the process, rendering the video. So from my PoV, the > version in bullseye is fine once this is fixed. > > Of course the latest and greatest if always better, but that's > incompatible with a stable release. We just have to cut it at some > point. > > While that's true, in this case I'm mostly concerned about all the > bugfixes that happened in the meantime. > > In any case, would you be interested in helping maintaining this > package? I don't have much time for that these days. I already maintain quite a few packages, I'm not sure I will be able to. Thanks for fixing this! signature.asc Description: PGP signature
Bug#985358: pitivi: A/V out of sync in rendered videos
On Tue, 2021-03-16 at 13:39 -0300, Antonio Terceiro wrote: As a user, I don't even know about all the stuff that could be better; pitivi worked ok for me as an amateur video editor. It just failed at the end of the process, rendering the video. So from my PoV, the version in bullseye is fine once this is fixed. Of course the latest and greatest if always better, but that's incompatible with a stable release. We just have to cut it at some point. While that's true, in this case I'm mostly concerned about all the bugfixes that happened in the meantime. In any case, would you be interested in helping maintaining this package? I don't have much time for that these days. signature.asc Description: This is a digitally signed message part
Bug#985358: pitivi: A/V out of sync in rendered videos
On Tue, Mar 16, 2021 at 04:45:39PM +0200, Sebastian Dröge wrote: > On Tue, 2021-03-16 at 11:16 -0300, Antonio Terceiro wrote: > > Dear Maintainer, > > The version of pitivi in bullseye is affected by the bug listed above: > rendered videos have A/V out of sync by a few seconds, while they sound > just find in the preview. > > I'm attaching the upstream patch that fixed the issue, already > backported to the current Debian package. I rebuilt pitivi locally with > this patch, and confirmed that it does fix the issue. > > Yeah I'm aware of this and various other issues in the version > currently in Debian, and which are fixed in the latest releases. > > Due to Debian release freeze policies the latest release is not > uploaded yet and I don't know how useful it is to just patch some of > the issues. > > In the end what we'd really want is the latest release. As a user, I don't even know about all the stuff that could be better; pitivi worked ok for me as an amateur video editor. It just failed at the end of the process, rendering the video. So from my PoV, the version in bullseye is fine once this is fixed. Of course the latest and greatest if always better, but that's incompatible with a stable release. We just have to cut it at some point. signature.asc Description: PGP signature
Bug#985358: pitivi: A/V out of sync in rendered videos
On Tue, 2021-03-16 at 11:16 -0300, Antonio Terceiro wrote: Dear Maintainer, The version of pitivi in bullseye is affected by the bug listed above: rendered videos have A/V out of sync by a few seconds, while they sound just find in the preview. I'm attaching the upstream patch that fixed the issue, already backported to the current Debian package. I rebuilt pitivi locally with this patch, and confirmed that it does fix the issue. Yeah I'm aware of this and various other issues in the version currently in Debian, and which are fixed in the latest releases. Due to Debian release freeze policies the latest release is not uploaded yet and I don't know how useful it is to just patch some of the issues. In the end what we'd really want is the latest release. signature.asc Description: This is a digitally signed message part
Bug#985358: pitivi: A/V out of sync in rendered videos
Package: pitivi Version: 2020.09.2-2 Severity: grave Tags: upstream patch Justification: renders package unusable Forwarded: https://gitlab.gnome.org/GNOME/pitivi/-/issues/2498 Dear Maintainer, The version of pitivi in bullseye is affected by the bug listed above: rendered videos have A/V out of sync by a few seconds, while they sound just find in the preview. I'm attaching the upstream patch that fixed the issue, already backported to the current Debian package. I rebuilt pitivi locally with this patch, and confirmed that it does fix the issue. -- System Information: Debian Release: bullseye/sid APT prefers unstable-debug APT policy: (500, 'unstable-debug'), (500, 'testing-debug'), (500, 'unstable'), (500, 'testing'), (1, 'experimental-debug'), (1, 'experimental') Architecture: amd64 (x86_64) Foreign Architectures: arm64 Kernel: Linux 5.10.0-4-amd64 (SMP w/4 CPU threads) Locale: LANG=pt_BR.UTF-8, LC_CTYPE=pt_BR.UTF-8 (charmap=UTF-8), LANGUAGE=pt_BR:pt:en Shell: /bin/sh linked to /usr/bin/dash Init: systemd (via /run/systemd/system) LSM: AppArmor: enabled Versions of packages pitivi depends on: ii gir1.2-gdkpixbuf-2.02.42.2+dfsg-1 ii gir1.2-ges-1.0 1.18.3-1 ii gir1.2-glib-2.0 1.66.1-1+b1 ii gir1.2-gst-plugins-bad-1.0 1.18.3-1+b1 ii gir1.2-gst-plugins-base-1.0 1.18.3-1 ii gir1.2-gstreamer-1.01.18.3-1 ii gir1.2-gtk-3.0 3.24.24-3 ii gir1.2-pango-1.01.46.2-3 ii gir1.2-peas-1.0 1.28.0-2+b1 pn gstreamer1.0-audiosink ii gstreamer1.0-gl [gstreamer1.0-videosink]1.18.3-1 ii gstreamer1.0-gtk3 [gstreamer1.0-videosink] 1.18.3-1 ii gstreamer1.0-plugins-bad [gstreamer1.0-videosink] 1.18.3-1+b1 ii gstreamer1.0-plugins-base 1.18.3-1 ii gstreamer1.0-plugins-good [gstreamer1.0-videosink] 1.18.3-1 ii gstreamer1.0-x [gstreamer1.0-videosink] 1.18.3-1 ii libc6 2.31-9 ii libcairo2 1.16.0-5 ii libglib2.0-02.66.7-2 ii libgstreamer1.0-0 1.18.3-1 ii libpython3.93.9.2-1 ii python3 3.9.2-2 ii python3-cairo 1.16.2-4+b2 ii python3-dbus1.2.16-5 ii python3-ges-1.0 1.18.3-1 ii python3-gi 3.38.0-2 ii python3-gi-cairo3.38.0-2 ii python3-gst-1.0 1.18.3-1 ii python3-matplotlib 3.3.4-1 ii python3-numpy 1:1.19.5-1 ii python3-xdg 0.27-2 ii python3.9 3.9.2-1 pitivi recommends no packages. Versions of packages pitivi suggests: pn frei0r-plugins ii gir1.2-gnomedesktop-3.03.38.4-1 pn gir1.2-gsound-1.0 ii gir1.2-notify-0.7 0.7.9-3 ii gstreamer1.0-libav 1.18.3-1 ii gstreamer1.0-plugins-ugly 1.18.3-1 -- no debconf information From: Thibault Saunier Date: Mon, 11 Jan 2021 17:50:26 -0300 Subject: pipeline: Avoid committing the timeline while rendering This can cause weird behavior and in particular it can lead to de-synchronized audio/video streams. Fixes https://gitlab.gnome.org/GNOME/pitivi/-/issues/2498 --- pitivi/editorperspective.py | 3 +++ pitivi/utils/pipeline.py| 8 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/pitivi/editorperspective.py b/pitivi/editorperspective.py index a25450c..49cbf61 100644 --- a/pitivi/editorperspective.py +++ b/pitivi/editorperspective.py @@ -128,6 +128,9 @@ class EditorPerspective(Perspective, Loggable): # Nothing to work with, Pitivi is starting up. return +if self.app.project_manager.current_project.pipeline.rendering(): +return + # Commit the timeline so its nested timelines assets are refreshed. ges_timeline.commit() diff --git a/pitivi/utils/pipeline.py b/pitivi/utils/pipeline.py index 95be106..c92c880 100644 --- a/pitivi/utils/pipeline.py +++ b/pitivi/utils/pipeline.py @@ -424,7 +424,7 @@ class SimplePipeline(GObject.Object, Loggable): Gst.debug_bin_to_dot_file_with_ts(self._pipeline, Gst.DebugGraphDetails.ALL, "pitivi.error") -if not self._rendering(): +if not self.rendering():