<div dir="ltr">Hi Dr. Thompson,<div><br></div><div>Really sorry for late reply. I have been overwhelmed by my siggraph aisa project recently. I changed the method to compute binomial coefficient from vtkMath::Binomial to vtkMath::Factorial using our memoization improvement and tried to merge from Lin.Ma/filter-bezier to Spline/filter-bezier.</div><div><br></div><div>What do you think is the proper thing to do next?</div><div><br></div><div>Best,</div><div>Lin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 20, 2015 at 9:12 AM, Ben Boeckel <span dir="ltr"><<a href="mailto:ben.boeckel@kitware.com" target="_blank">ben.boeckel@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Apr 17, 2015 at 22:53:50 -0400, David Thompson wrote:<br>
> Thanks for pushing the branch to gitlab.<br>
><br>
> > Do I need to make a merge requests for that? There are still many<br>
> > things to be improved in the code.<br>
><br>
> You can submit merge requests -- but not to vtk/vtk. I have created a<br>
> new group named splines and added you as a developer. Please fork this<br>
> repository:<br>
><br>
>     <a href="https://gitlab.kitware.com/splines/vtk" target="_blank">https://gitlab.kitware.com/splines/vtk</a><br>
<br>
</span>I have local (largely untested, but a smoke test succeeded) changes<br>
which allow submitting MRs to other forks of the same project, so this<br>
whole repo shuffle thing should be unnecessary once that gets some more<br>
testing (and we should finally have a complete end-to-end test instance<br>
up and running here this week).<br>
<span class=""><br>
> You will have to tell gitlab to delete your existing fork of VTK (only<br>
> on the web, not the files on your computer!) and fork that repository<br>
> instead in order to submit merge requests. To remove your current fork<br>
> of VTK, visit:<br>
><br>
>     <a href="https://gitlab.kitware.com/Lin.Ma/vtk/edit" target="_blank">https://gitlab.kitware.com/Lin.Ma/vtk/edit</a><br>
><br>
> Scroll to the bottom of the page and click the red "Remove project"<br>
> button. Then you can fork splines/vtk by visiting<br>
><br>
>     <a href="https://gitlab.kitware.com/splines/vtk" target="_blank">https://gitlab.kitware.com/splines/vtk</a><br>
><br>
> and clicking its "Fork" button at the top right of the page. You will<br>
> see that splines/vtk already contains your filters-bezier branch and<br>
> that I have made some changes (so that it meets the VTK style<br>
> guidelines).<br>
<br>
</span>Probably easier to just rename the project rather than deleting it.<br>
Alternatively, give developer access to Lin.Ma for splines/vtk and make<br>
an MR from splines/vtk to itself.<br>
<span class=""><br>
> Please review the changes and make sure you understand what I've done.<br>
> If you are unsure what I did or why I did it, please ask.<br>
><br>
> Once you have reviewed the changes, please make a small change and<br>
> submit a merge request from Lin.Ma/vtk filters-bezier to vtk/splines<br>
> filters-bezier to test that merge requests work.<br>
<br>
</span>FYI, kwrobot and buildbot do not act on reporitories other than the ones<br>
specified in their scripts, so your @buildbot test I saw will go<br>
ignored until it shows up as an MR on vtk/vtk.<br>
<span class="HOEnZb"><font color="#888888"><br>
--Ben<br>
</font></span></blockquote></div><br></div>