[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2020-05-07 Thread Boudewijn Rempt
https://bugs.kde.org/show_bug.cgi?id=396128

Boudewijn Rempt  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/kde/ |https://invent.kde.org/kde/
   |krita/commit/f5ac73ca0e1a34 |krita/commit/61c79aacb81653
   |a508ac74831afc7ea44bbf3a55  |527817157dd108c46d6d048c73

--- Comment #10 from Boudewijn Rempt  ---
Git commit 61c79aacb81653527817157dd108c46d6d048c73 by Boudewijn Rempt, on
behalf of Emmet O'Neill.
Committed on 07/05/2020 at 06:48.
Pushed by rempt into branch 'krita/4.3'.

Fix FFMPEG issue associated with even:odd anims.

Quick fix for a bug causing animations with one odd dimension
to fail to render and report an unkown error. FFMPEG requires
even render dimensions, and fixing that in even:odd cases creates
a minor shift in aspect ratio, causing animation rendering to fail
while preserve aspect ratio is set.
(cherry picked from commit f5ac73ca0e1a34a508ac74831afc7ea44bbf3a55)

M  +25   -31   plugins/extensions/animationrenderer/AnimationRenderer.cpp
M  +3-1plugins/extensions/animationrenderer/AnimationRenderer.h
M  +0-4plugins/extensions/animationrenderer/DlgAnimationRenderer.cpp
M  +2-4plugins/extensions/animationrenderer/video_saver.cpp

https://invent.kde.org/kde/krita/commit/61c79aacb81653527817157dd108c46d6d048c73

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2020-05-06 Thread Emmet O'Neill
https://bugs.kde.org/show_bug.cgi?id=396128

Emmet O'Neill  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED
  Latest Commit||https://invent.kde.org/kde/
   ||krita/commit/f5ac73ca0e1a34
   ||a508ac74831afc7ea44bbf3a55

--- Comment #9 from Emmet O'Neill  ---
Git commit f5ac73ca0e1a34a508ac74831afc7ea44bbf3a55 by Emmet O'Neill.
Committed on 06/05/2020 at 21:48.
Pushed by emmetoneill into branch 'master'.

Fix FFMPEG issue associated with even:odd anims.

Quick fix for a bug causing animations with one odd dimension
to fail to render and report an unkown error. FFMPEG requires
even render dimensions, and fixing that in even:odd cases creates
a minor shift in aspect ratio, causing animation rendering to fail
while preserve aspect ratio is set.

M  +25   -31   plugins/extensions/animationrenderer/AnimationRenderer.cpp
M  +3-1plugins/extensions/animationrenderer/AnimationRenderer.h
M  +0-4plugins/extensions/animationrenderer/DlgAnimationRenderer.cpp
M  +2-4plugins/extensions/animationrenderer/video_saver.cpp

https://invent.kde.org/kde/krita/commit/f5ac73ca0e1a34a508ac74831afc7ea44bbf3a55

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2020-04-15 Thread Tymond
https://bugs.kde.org/show_bug.cgi?id=396128

Tymond  changed:

   What|Removed |Added

 Status|REOPENED|ASSIGNED

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2020-04-13 Thread Emmet O'Neill
https://bugs.kde.org/show_bug.cgi?id=396128

Emmet O'Neill  changed:

   What|Removed |Added

 CC||emmetoneill@gmail.com
   Assignee|krita-bugs-n...@kde.org |emmetoneill@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2019-12-06 Thread Tymond
https://bugs.kde.org/show_bug.cgi?id=396128

--- Comment #8 from Tymond  ---
Also whoever is fixing this, please take note of bug 412599.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2019-12-06 Thread Tymond
https://bugs.kde.org/show_bug.cgi?id=396128

Tymond  changed:

   What|Removed |Added

 Resolution|FIXED   |---
 CC||tamtamy.tym...@gmail.com
 Status|RESOLVED|REOPENED

--- Comment #7 from Tymond  ---
Because ffmpeg complains also when just the canvas (frames images) are in
uneven size, Krita should show an error in this case, too.

Steps to reproduce:
1. Make a canvas 480x529. 
2. Create a new frame on a layer.
3. In Render Animation dialog, choose size 480x530.

OBSERVED RESULTS:
4. It won't render and it will say "Unknown Error". In log_encode.log file it
will complain about size being not even.

EXPECTED:
This situation should be impossible to reach. Krita should show an error
instead of giving ffmpeg those values.

NOTE:
I'm not sure if ffmpeg should complain in this case, but latest version (4.2.1)
complains, and last time I spent 1h on figuring out what went wrong, so please
let's just make it into an error message. Maybe some time in the future it can
be removed.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2019-03-03 Thread wolthera
https://bugs.kde.org/show_bug.cgi?id=396128

wolthera  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #6 from wolthera  ---
I think I can close this now.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2018-07-03 Thread Scott Petrovic
https://bugs.kde.org/show_bug.cgi?id=396128

--- Comment #5 from Scott Petrovic  ---
Does it give you an issue if you render it out at 2690x2303 (the original
size)...or only if you change the dimensions in the render dialog?

Maybe we are both referring to different things when we say 'custom
dimensions'. Custom dimensions to me means exporting a video at different size
then your canvas size.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2018-07-03 Thread wolthera
https://bugs.kde.org/show_bug.cgi?id=396128

--- Comment #4 from wolthera  ---
2690x2303 and the render animation dialog had 800x684 for width/height.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2018-07-03 Thread Scott Petrovic
https://bugs.kde.org/show_bug.cgi?id=396128

--- Comment #3 from Scott Petrovic  ---
What dimensions is giving you the error? I wasn't aware of any issues with
exporting out at a normal 100%

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2018-07-03 Thread wolthera
https://bugs.kde.org/show_bug.cgi?id=396128

--- Comment #2 from wolthera  ---
It is not just custom dimensions, in my case the canvas itself had an even
number of pixels for the width and an odd number for the height, which caused
the error.

-- 
You are receiving this mail because:
You are watching all bug changes.

[krita] [Bug 396128] mp4 cannot be encoded if either of the frame dimensions is not divisable by 2

2018-07-03 Thread Scott Petrovic
https://bugs.kde.org/show_bug.cgi?id=396128

Scott Petrovic  changed:

   What|Removed |Added

 CC||scottpetro...@gmail.com
 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED

--- Comment #1 from Scott Petrovic  ---
This is a problem that exists when we allow people to manually enter different
dimensions for exporting. Some dimenions just won't be valid depending on what
people enter. This is a situation with how ffmpeg works and ffmpeg doesn't seem
to handle errors very well with it. Other programs like Blender handle it
equally as bad (just errors without any message).

We might have to add custom validation logic before exporting to see if the
dimensions are divisible by 2. Maybe we don't allow the export button to be
pressed if it is invalid and show a label on the UI with a message.

Not sure if anyone else has ideas. I think we are going to have to do something
though in Krita code...not something ffmpeg will be able to tell us.

-- 
You are receiving this mail because:
You are watching all bug changes.