[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-21 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

 Status|REPORTED|NEEDSINFO
 Resolution|--- |WAITINGFORINFO
 CC||dimul...@gmail.com

--- Comment #1 from Dmitry Kazakov  ---
Hi, Nathur!

Could you please check this build when it is finished? I think I have fixed
this bug:

https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/1987/

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-21 Thread Nathan U.
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #2 from Nathan U.  ---
(In reply to Dmitry Kazakov from comment #1)
> Hi, Nathur!
> 
> Could you please check this build when it is finished? I think I have fixed
> this bug:
> 
> https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/1987/

Thanks Dmitry!

After running some tests with this new build, I can say... I'm confused as hell

In some files I get the Unknown Error, in some I don't and the animation
renders as intended

I've zero idea what could be causing this whole thing, but eventually I thought
to check the log files that came with the rendered animations, and found this:

> D:/krita-nightly-x64-5.2.0-rc1-4cc1ab5a65/bin/ffmpeg.exe -hide_banner -y -y 
> -r 30 -i D:/Mis Cosas/Imagenes/Por mí/Hollow Knight/Grimm 
> Dross/fotograma%04d.png -ss 0:0:20.889 -t 0:0:6.006 -i D:/Mis 
> Cosas/Imagenes/Por mí/Hollow Knight/Grimm Dross/GrimmDross_Audio.wav -c:v 
> libopenh264 -b:v 5000k D:/Mis Cosas/Imagenes/Por mí/Hollow Knight/Grimm 
> Dross/GrimmDross_2.mp4
> =
> [image2 @ 018fd2618240] Could find no file with path 'D:/Mis 
> Cosas/Imagenes/Por mí/Hollow Knight/Grimm Dross/fotograma%04d.png' and index 
> in the range 0-4
> D:/Mis Cosas/Imagenes/Por mí/Hollow Knight/Grimm Dross/fotograma%04d.png: No 
> such file or directory

Hope it helps

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-21 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=474679

Bug Janitor Service  changed:

   What|Removed |Added

 Resolution|WAITINGFORINFO  |---
 Status|NEEDSINFO   |REPORTED

--- Comment #3 from Bug Janitor Service  ---
Thanks for your comment!

Automatically switching the status of this bug to REPORTED so that the KDE team
knows that the bug is ready to get confirmed.

In the future you may also do this yourself when providing needed information.

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-22 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #5 from Dmitry Kazakov  ---
Hi, Nathan!

Could you please tell me what files you have in this folder:

D:/Mis Cosas/Imagenes/Por mí/Hollow Knight/Grimm Dross/

There should be files like fotograma0001.png fotograma0002.png and so on...

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-22 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #4 from Dmitry Kazakov  ---
Hi, Nathan!

Could you please tell me what files you have in this folder:

D:/Mis Cosas/Imagenes/Por mí/Hollow Knight/Grimm Dross/

There should be files like fotograma0001.png fotograma0002.png and so on...

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-22 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #6 from Dmitry Kazakov  ---
Okay, I cannot reproduce this bug locally, however hard I try. Please send me
the list of files you have in this folder, or, what are the numbers in files
'fotograma%04d.png' in that folder.

Btw, could you also check if you have enough space on drive D:? Perhaps Krita
fails to write all the frames there?

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-22 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 Status|REPORTED|NEEDSINFO

--- Comment #7 from Dmitry Kazakov  ---
Marking the bug as waiting-for-info

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-22 Thread Nathan U.
https://bugs.kde.org/show_bug.cgi?id=474679

Nathan U.  changed:

   What|Removed |Added

 Resolution|WAITINGFORINFO  |---
 Status|NEEDSINFO   |REPORTED

--- Comment #8 from Nathan U.  ---
(In reply to Dmitry Kazakov from comment #6)
> Okay, I cannot reproduce this bug locally, however hard I try. Please send
> me the list of files you have in this folder, or, what are the numbers in
> files 'fotograma%04d.png' in that folder.
> 
> Btw, could you also check if you have enough space on drive D:? Perhaps
> Krita fails to write all the frames there?

Hey Dmitry!

No, it says I've got around 300 GB of free space in D:, so it can't be that

So, I went and tried to render the GrimmDross animation again, and found that
the frame files, rather than 0001, 0002, etc. were instead starting from 0633
(ending at 0814). Likely given that originally the keyframes were around that
number in the .kra file. But thing is, I'd moved the animation to frame 0 a
while ago

I also checked with a test animation I have, and likewise the frame files start
at 0024, corresponding with the animation in the .kra file (and ending at
0084). But for some reason, when I click OK on the error popup, rather than
remove all the frame files, it leaves 0024-0047 intact, which matches the
duration of the first keyframe

Later though, I thought maybe the error was because one of the folders in the
path has an accented letter (í), so I tried rendering it in a different folder,
but I still get the error

Additionally, I tried setting the Animation Cache to "on-disk", but still
nothing

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-25 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #9 from Dmitry Kazakov  ---
Adding an  accented letter to the path was the first thing I tried to do here
locally :)

Could you give a bit more info about what you mean by "I'd moved the animation
to frame 0 a while ago"? Just removed a few of the leading columns from the
timeline?

Btw, is it possible to share the crashing image with me (privately, via email),
so I could test it better?

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-25 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #10 from Dmitry Kazakov  ---
Git commit 0ed384a5dae0d99582f964c0061519769b7a0176 by Dmitry Kazakov.
Committed on 25/09/2023 at 15:46.
Pushed by dkazakov into branch 'master'.

Fix removal of duplicated frames in "Only Unique Frames" mode

The offset was applied incorrectly so the clean-up stage of the rendering
process removed incorrect files (basically, the files without the
normalizing offset)

The patch should also fix a possible offset issue between the video
and audio when rendering multiple times into the same folder
Related: bug 450449, bug 458997

M  +2-2libs/ui/KisAsyncAnimationFramesSavingRenderer.cpp
M  +1-17   libs/ui/animation/KisAnimationRender.cpp
M  +0-4libs/ui/animation/KisAnimationRender.h
M  +13   -21   libs/ui/dialogs/KisAsyncAnimationFramesSaveDialog.cpp
M  +2-2libs/ui/dialogs/KisAsyncAnimationFramesSaveDialog.h

https://invent.kde.org/graphics/krita/-/commit/0ed384a5dae0d99582f964c0061519769b7a0176

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-25 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #11 from Dmitry Kazakov  ---
Git commit 2fbd233de9554bb94a98fbf53cb14dc3e457fe5d by Dmitry Kazakov.
Committed on 25/09/2023 at 15:50.
Pushed by dkazakov into branch 'krita/5.2'.

Fix removal of duplicated frames in "Only Unique Frames" mode

The offset was applied incorrectly so the clean-up stage of the rendering
process removed incorrect files (basically, the files without the
normalizing offset)

The patch should also fix a possible offset issue between the video
and audio when rendering multiple times into the same folder
Related: bug 450449, bug 458997

M  +2-2libs/ui/KisAsyncAnimationFramesSavingRenderer.cpp
M  +1-17   libs/ui/animation/KisAnimationRender.cpp
M  +0-4libs/ui/animation/KisAnimationRender.h
M  +13   -21   libs/ui/dialogs/KisAsyncAnimationFramesSaveDialog.cpp
M  +2-2libs/ui/dialogs/KisAsyncAnimationFramesSaveDialog.h

https://invent.kde.org/graphics/krita/-/commit/2fbd233de9554bb94a98fbf53cb14dc3e457fe5d

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-25 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 Status|REPORTED|NEEDSINFO

--- Comment #12 from Dmitry Kazakov  ---
Hi, Nathan!

Could you please test this build when it completes building? I couldn't
reproduce the original issue, but I could reproduce a related issue with
"unique frames" handling. Perhaps it could also fix your issue?

This is the link to the build (it should complete in half an hour):
https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/1992/

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-25 Thread Nathan U.
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #13 from Nathan U.  ---
(In reply to Dmitry Kazakov from comment #9)
> Adding an  accented letter to the path was the first thing I tried to do
> here locally :)
> 
> Could you give a bit more info about what you mean by "I'd moved the
> animation to frame 0 a while ago"? Just removed a few of the leading columns
> from the timeline?
> 
> Btw, is it possible to share the crashing image with me (privately, via
> email), so I could test it better?

Hey Dmitry!

By "moving the animation to frame 0" I meant dragging the entire set of
keyframes back to the very beginning of the timeline, with the first keyframe
being in 0 and so on. Because when I'd first tried to render the animation it
began around the 600 mark, in order to match the original audio file I was
working with

Also by "crashing image" you mean the .kra files that won't render, right?

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-25 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=474679

Bug Janitor Service  changed:

   What|Removed |Added

 Status|NEEDSINFO   |REPORTED
 Resolution|WAITINGFORINFO  |---

--- Comment #14 from Bug Janitor Service  ---
Thanks for your comment!

Automatically switching the status of this bug to REPORTED so that the KDE team
knows that the bug is ready to get confirmed.

In the future you may also do this yourself when providing needed information.

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-09-26 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

 Status|REPORTED|NEEDSINFO
 Resolution|--- |WAITINGFORINFO

--- Comment #15 from Dmitry Kazakov  ---
Hi, Nathan!

Yes, by "crashing image" I mean the image that causes the problem. You can
share it privately via email if you don't want to have it publicly available.

You can also try the new build, perhaps it has the issue fixed... 
https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/1992/

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-10 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #16 from Bug Janitor Service  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-11 Thread Nathan U.
https://bugs.kde.org/show_bug.cgi?id=474679

Nathan U.  changed:

   What|Removed |Added

 Status|NEEDSINFO   |REPORTED
 Resolution|WAITINGFORINFO  |---

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-17 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|ASSIGNED
   Assignee|krita-bugs-n...@kde.org |dimul...@gmail.com

--- Comment #17 from Dmitry Kazakov  ---
Okay, I have the testing file on my email, so I will look into that.

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-17 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://invent.kde.org/grap
   ||hics/krita/-/commit/2cf4dfb
   ||f0de99d81fbe32654d0fad6a2d6
   ||29ad16

--- Comment #18 from Dmitry Kazakov  ---
Git commit 2cf4dfbf0de99d81fbe32654d0fad6a2d629ad16 by Dmitry Kazakov.
Committed on 17/10/2023 at 16:42.
Pushed by dkazakov into branch 'krita/5.2'.

Fix rendering of the video when start frame is non-zero

When start frame is non-zero or if the frames are saved with
numbering offset, we should pass correct offsets to ffmpeg.

Before the patch, it had the following problems:

1) If start numbering was > 4, then ffmpeg rendering failed.

2) If first frame was non-zero, then audio was saved with wrong
   offset

M  +7-6libs/ui/animation/KisVideoSaver.cpp

https://invent.kde.org/graphics/krita/-/commit/2cf4dfbf0de99d81fbe32654d0fad6a2d629ad16

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-17 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

Dmitry Kazakov  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap
   |hics/krita/-/commit/2cf4dfb |hics/krita/-/commit/a11cf8e
   |f0de99d81fbe32654d0fad6a2d6 |076e537bb457a8226cf4703cbeb
   |29ad16  |33cde0

--- Comment #19 from Dmitry Kazakov  ---
Git commit a11cf8e076e537bb457a8226cf4703cbeb33cde0 by Dmitry Kazakov.
Committed on 17/10/2023 at 16:43.
Pushed by dkazakov into branch 'master'.

Fix rendering of the video when start frame is non-zero

When start frame is non-zero or if the frames are saved with
numbering offset, we should pass correct offsets to ffmpeg.

Before the patch, it had the following problems:

1) If start numbering was > 4, then ffmpeg rendering failed.

2) If first frame was non-zero, then audio was saved with wrong
   offset

M  +7-6libs/ui/animation/KisVideoSaver.cpp

https://invent.kde.org/graphics/krita/-/commit/a11cf8e076e537bb457a8226cf4703cbeb33cde0

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-17 Thread Dmitry Kazakov
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #20 from Dmitry Kazakov  ---
Hi, Nathan!

Could you please test this build when it is finished?

https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/2015/

I think the bug is fixed now. Please also check if there is any offset between
the video and audio streams. Now there should be no offset :)

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

[krita] [Bug 474679] Krita fails to render animations that don't have starting frame as 0

2023-10-19 Thread Nathan U.
https://bugs.kde.org/show_bug.cgi?id=474679

--- Comment #21 from Nathan U.  ---
(In reply to Dmitry Kazakov from comment #20)
> Hi, Nathan!
> 
> Could you please test this build when it is finished?
> 
> https://binary-factory.kde.org/job/Krita_Stable_Windows_Build/2015/
> 
> I think the bug is fixed now. Please also check if there is any offset
> between the video and audio streams. Now there should be no offset :)

Dmitry, you bloody genius, it works!! Thank you so much for bearing with all
this

(Also the offset seems to be alright)

There *are* some other bugs I've found, but I'll get those handled in separate
reports. Again, thank you!

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