On Sat, Jul 17, 2010 at 12:13 AM, Aaron S. Meurer <asmeu...@gmail.com> wrote: > > On Jul 16, 2010, at 4:00 AM, smichr wrote: > >> Hi all, >> >> I have a lot of work that I have separated out into about 35 issues. >> The work deals with everything from minor editing of code, to handling >> of non-commutatives in simplifications and (hopefully) a total purge >> of power-rule infractions (e.g. allowing sqrt(x*y) to become >> sqrt(x)*sqrt(y)). When all changes are made, the runtime of the test >> suite drops by about 20%.[1] >> >> This work has survived through the new polys and the ploughing of the >> core. If it's going to make it in I would really like to get some help >> on reviewing these issues before any other major changes take place. >> >> ======== it's in branch t at smichr's' github >> account =========== >> >> Everything is sitting in branch t at the smichr account at github. >> Everything has been rebased on master. All tests and doctests passed >> in 64 bit (but I am double checking after the rebase on the current >> master). >> >> Although the first few commits are independent of each other the >> commits that follow them become interdependent. I'm not sure what the >> best way is to review these. I can' pull them off independently for >> review since they won't rebase on master as independent; the term we >> use on the issue page for such issues is "blocked on". The way I would >> review these if they were on someone else's branch would be to use >> gitk to view a single commit; if I wanted to confirm that the tests >> passed I would checkout the particular commit and test it. >> >> So, >> >> 1) are there ideas of how best to review this? > > Also, it would help if you could avoid rebasing a branch once it is up for > review. Basically, if I add a comment to your code in GitHub, then you > rebase, you won't see the comment in your branch. I will still show up in > http://github.com/smichr/sympy/comments, but on the commit object that I > originally commented on. > > I hope to go through and do just that this weekend.
Thanks Aaron. Ondrej -- You received this message because you are subscribed to the Google Groups "sympy" group. To post to this group, send email to sy...@googlegroups.com. To unsubscribe from this group, send email to sympy+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/sympy?hl=en.