Bug 423439 - Animated .gif Rendering output size fix is not in the 4.3.1 prealpha appimage
Summary: Animated .gif Rendering output size fix is not in the 4.3.1 prealpha appimage
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Animation (show other bugs)
Version: nightly build (please specify the git hash!)
Platform: Debian stable Linux
: NOR minor
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-24 11:36 UTC by Ahab Greybeard
Modified: 2020-06-24 11:53 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahab Greybeard 2020-06-24 11:36:55 UTC
SUMMARY
4.2.9 and 4.3.0 and 23 Jun 4.3.1 prealpha (git 5e750g3) appimages have non-working Width: and Height: controls in the Rendering output window when using .gif as the output option. The rendered output file is always at the same size as the animated image.
(It works fine for .mp4 output.)

The 23 Jun 5.0.0 prealpha appimage (git 96e3af2) has properly working Width: and Height: controls for animated .gif.

Has a fix not been incorporated into the 4.3.1 pre-alpha?
Comment 1 Halla Rempt 2020-06-24 11:49:11 UTC
Could be... Would that be this commit: 

commit 3efcd3e648c62b017b3146b9efb6826d6d1aaa0d
Author: Ivan Yossi <ghevan@gmail.com>
Date:   Mon Apr 27 02:07:01 2020 -0500

    Fix ffmpeg argument construct for GIF save
    
    BUG:420215

diff --git a/plugins/extensions/animationrenderer/video_saver.cpp b/plugins/extensions/animationrenderer/video_saver.cpp
index 2394d54e2b..d2aa947a58 100644
--- a/plugins/extensions/animationrenderer/video_saver.cpp
+++ b/plugins/extensions/animationrenderer/video_saver.cpp
@@ -261,14 +261,18 @@ KisImportExportErrorCode VideoSaver::encode(const QString &savedFilesMask, const
                  << "-start_number" << QString::number(clipRange.start())
                  << "-i" << savedFilesMask
                  << "-i" << palettePath
-                 << "-lavfi" << "[0:v][1:v] paletteuse"
-                 << "-y" << resultFile;
+                 << "-lavfi";
+
+            QString filterArgs;
 
             // if we are exporting out at a different image size, we apply scaling filter
             if (m_image->width() != options.width || m_image->height() != options.height) {
-                args << "-vf" << exportDimensions;
+                filterArgs.append(exportDimensions + "[0:v];");
             }
 
+            args << filterArgs.append("[0:v][1:v] paletteuse")
+                 << "-y" << resultFile;
+
 
             dbgFile << "savedFilesMask" << savedFilesMask << "start" << QString::number(clipRange.start()) << "duration" << clipRange.duration();
 
lines 1-34/34 (END)

?
Comment 2 Halla Rempt 2020-06-24 11:53:09 UTC
Git commit 7647f6529acd6cbba6a455c04822ca8d610f3687 by Boudewijn Rempt, on behalf of Ivan Yossi.
Committed on 24/06/2020 at 11:52.
Pushed by rempt into branch 'krita/4.3'.

Fix ffmpeg argument construct for GIF save
Related: bug 420215
(cherry picked from commit 3efcd3e648c62b017b3146b9efb6826d6d1aaa0d)

M  +7    -3    plugins/extensions/animationrenderer/video_saver.cpp

https://invent.kde.org/graphics/krita/commit/7647f6529acd6cbba6a455c04822ca8d610f3687