[Kdenlive-devel] Git commit 66516b9 - Vincent Pinion

[Evert Vorster]

Hi there...

This commit unfortunately has made my life quite a bit more difficult
than it needs to be.
I submitted a bug about this two weeks ago, (0003323), but it has not
been assigned to anyone yet.

In short, it's far more common to do audio fades than fade to and from
black on video clips.

I can see that it makes sense to do video stuff on video clips, and
audio stuff on audio clips.

However, when I split out the audio out of the video clips as a
compromise, I am still unable to quickly add audio fades, as the
functionality is broken.

I do realize that there is a massive code cleanup effort underway, and
that messing with this code is as much fun as getting poked in the
eye.

I just prefer the previous behavior, and if it's too much trouble to
make audio fades work as expected on audio clips that were split from
video clips, I would rather see this commit reverted.

Kind regards,

[Vincent Pinon]

Hello,

I'm so sorry Evert to have caused trouble with my mistake and even not
replying to your request.
I've fixed the thing last week but didn't push the change since then :-/

I plan to take some time this week to fix the little things on 0.9.8 raised on
mantis; if you have priorities to raise that would help me to do useful
things:-)

Post by Evert Vorster
Hi there...
This commit unfortunately has made my life quite a bit more difficult
than it needs to be.
I submitted a bug about this two weeks ago, (0003323), but it has not
been assigned to anyone yet.
In short, it's far more common to do audio fades than fade to and from
black on video clips.
I can see that it makes sense to do video stuff on video clips, and
audio stuff on audio clips.
However, when I split out the audio out of the video clips as a
compromise, I am still unable to quickly add audio fades, as the
functionality is broken.
I do realize that there is a massive code cleanup effort underway, and
that messing with this code is as much fun as getting poked in the
eye.
I just prefer the previous behavior, and if it's too much trouble to
make audio fades work as expected on audio clips that were split from
video clips, I would rather see this commit reverted.
Kind regards,

[Evert Vorster]

Thanks for both replies. I'll try "next"

According to your mail to the list next is supposed to become master anyways.

I'll keep on writing bugs in Mantis. There are no real show-stoppers,
so pick whatever low hanging fruit you see. :)

-Evert-

Post by Vincent Pinon
Hello,
I'm so sorry Evert to have caused trouble with my mistake and even not
replying to your request.
I've fixed the thing last week but didn't push the change since then :-/
I plan to take some time this week to fix the little things on 0.9.8 raised on
mantis; if you have priorities to raise that would help me to do useful
things:-)

Post by Evert Vorster
Hi there...
This commit unfortunately has made my life quite a bit more difficult
than it needs to be.
I submitted a bug about this two weeks ago, (0003323), but it has not
been assigned to anyone yet.
In short, it's far more common to do audio fades than fade to and from
black on video clips.
I can see that it makes sense to do video stuff on video clips, and
audio stuff on audio clips.
However, when I split out the audio out of the video clips as a
compromise, I am still unable to quickly add audio fades, as the
functionality is broken.
I do realize that there is a massive code cleanup effort underway, and
that messing with this code is as much fun as getting poked in the
eye.
I just prefer the previous behavior, and if it's too much trouble to
make audio fades work as expected on audio clips that were split from
video clips, I would rather see this commit reverted.
Kind regards,

------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
Kdenlive-devel mailing list
https://lists.sourceforge.net/lists/listinfo/kdenlive-devel

[Evert Vorster]

I'm trying next at the moment.

The issue with adding audio is definitely fixed, you can mark that bug as done.

However, now I am missing the "Lossless" profiles. That might be
something to cherry-pick from the current master....

-Evert-

Post by Evert Vorster
Thanks for both replies. I'll try "next"
According to your mail to the list next is supposed to become master anyways.
I'll keep on writing bugs in Mantis. There are no real show-stoppers,
so pick whatever low hanging fruit you see. :)
-Evert-

Post by Vincent Pinon
Hello,
I'm so sorry Evert to have caused trouble with my mistake and even not
replying to your request.
I've fixed the thing last week but didn't push the change since then :-/
I plan to take some time this week to fix the little things on 0.9.8 raised on
mantis; if you have priorities to raise that would help me to do useful
things:-)

Post by Evert Vorster
Hi there...
This commit unfortunately has made my life quite a bit more difficult
than it needs to be.
I submitted a bug about this two weeks ago, (0003323), but it has not
been assigned to anyone yet.
In short, it's far more common to do audio fades than fade to and from
black on video clips.
I can see that it makes sense to do video stuff on video clips, and
audio stuff on audio clips.
However, when I split out the audio out of the video clips as a
compromise, I am still unable to quickly add audio fades, as the
functionality is broken.
I do realize that there is a massive code cleanup effort underway, and
that messing with this code is as much fun as getting poked in the
eye.
I just prefer the previous behavior, and if it's too much trouble to
make audio fades work as expected on audio clips that were split from
video clips, I would rather see this commit reverted.
Kind regards,

------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
Kdenlive-devel mailing list
https://lists.sourceforge.net/lists/listinfo/kdenlive-devel

--
Evert Vorster
Chief Observer
WG Cook