Patches to fix race conditions in auto-alignment feature

[farid abdelnour]

Greetings

2017-04-14 17:08 GMT-03:00 Raphaël Marinier <raphael.marinier at gmail.com>:

> Thanks for the warm welcome.
>

Happy to have you here.


> I posted the patches to phabricator as advised.
>

Perfect.


> My patch is not a response to the March status update, but I read it
> and it was good to see that the project was actively being developed
> and to get an idea where it's heading.
>
> I could offer some help to develop Kdenlive, e.g. starting with fixing
> bugs or adding small features. At this point, I would not commit to
> the bigger tasks mentioned in the status update.
>

We have a list of low hanging bugs [1] if you'd like to cherry pick one.
Avoid timeline related issues since those will go away after the
refactoring. But feel free to also browse through the bug tracker or
phabricator's feature proposals [2][3][4] in case you find something more
interesting to work on.


> Raphael
>

Happy hacking

[1]
https://bugs.kde.org/buglist.cgi?quicksearch=product%3Akdenlive%20flag%3Alow_hanging&list_id=1434110
[2] https://phabricator.kde.org/T3020
[3] https://phabricator.kde.org/T2336
[4] https://phabricator.kde.org/T5434


>
> On Fri, Apr 14, 2017 at 4:50 PM, farid abdelnour <snd.noise at gmail.com>
> wrote:
> > Hi Raphaël,
> >
> > Welcome to the community!
> >
> > Is your patch a response to the recent post we published [1] or did you
> get
> > here by other means?
> >
> > Over at phabricator you'll also be able to get a glimpse of the
> refactoring
> > work being made, exiting times coming to Kdenlive.
> >
> > [1] https://kdenlive.org/2017/03/kdenlive-status-update/
> >
> > Thanks for the contribution :)
> >
> > Cheers
> >
> > 2017-04-14 6:55 GMT-03:00 alcinos <french.ebook.lover at gmail.com>:
> >>
> >> Hi Raphaël,
> >>
> >> Thank you for your contribution! Your PR looks good to me so far.
> >>
> >> Please note however that KDE (and thus Kdenlive) uses Phabricator for
> the
> >> review and merge process (github is only a mirror). Would it be
> possible for
> >> you to submit it there? (How to:
> >> https://community.kde.org/Infrastructure/Phabricator#Posting_a_Patch,
> link
> >> to the phabricator: https://phabricator.kde.org/)
> >> You can set mardelle and alcinos as reviewers.
> >>
> >> Cheers,
> >>
> >> alcinos
> >>
> >>
> >>
> >> 2017-04-13 23:23 GMT+02:00 Raphaël Marinier <raphael.marinier at gmail.com
> >:
> >>>
> >>> Hi all,
> >>>
> >>> I am a new user of kdenlive, and noticed arbitrary alignment of tracks
> >>> from time to time when using the auto alignment feature.
> >>> I investigated and found one race condition (and a couple of smaller
> >>> issues). I propose a fix with pull request
> >>> https://github.com/KDE/kdenlive/pull/6.
> >>>
> >>> Please have a look and tell me what you think,
> >>>
> >>> Thanks,
> >>>
> >>> Raphael M.
> >>
> >>
> >
> >
> >
> > --
> > 1111.1010.r.i.1101|n.o.i.s.1110|i.m.1010.g.1110|مقاومة
> > fsf member #5439
> > usuario GNU/Linux #471966
> > |_|0|_|
> > |_|_|0|
> > |0|0|0|
> > <a href="http://www.gunga.com.br">gunga</a>
> > <a href="http://www.tempoecoarte.com.br">tempoecoarte</a>
> > <a href="http://www.atelier-labs.org">atelier-labs</a>
> > <a href="http://www.mocambos.net">rede mocambos</a>
>



-- 
1111.1010.r.i.1101|n.o.i.s.1110|i.m.1010.g.1110|مقاومة
fsf member #5439
usuario GNU/Linux #471966
|_|0|_|
|_|_|0|
|0|0|0|
<a href="http://www.gunga.com.br">gunga</a>
<a href="http://www.tempoecoarte.com.br">tempoecoarte</a>
<a href="http://www.atelier-labs.org">atelier-labs</a>
<a href="http://www.mocambos.net">rede mocambos</a>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20170415/9f12aabc/attachment.html>