I'll look over the referee patch. Andrey
On Jan 21, 10:50 am, mhampton <hampto...@gmail.com> wrote: > OK, I added a referee patch. > > -Marshall > > On Jan 21, 9:38 am, William Stein <wst...@gmail.com> wrote: > > > On Thu, Jan 21, 2010 at 7:32 AM, mhampton <hampto...@gmail.com> wrote: > > > I would like to encourage people to review ticket 7109: > > > >http://trac.sagemath.org/sage_trac/ticket/7109 > > > > This was originally submitted by Volker Braun, and I think it is his > > > first contribution to Sage. It is an important refactoring of the > > > polyhedra code in geometry/polyhedra.py. This rewrite is much > > > cleaner, easier to extend in the future, and fixes a number of bugs in > > > my original code (for example: tickets 6915, 6214, and 5134 will all > > > be closed by this I think). > > > > Because of some lack of doctests and breaks in backwards- > > > compatibility, I added and changed a substantial amount to Volker's > > > original patch. But now I feel I shouldn't give it a positive > > > review. So I am asking that someone else check it out. > > > > I don't think you need to be an expert in polyhedra, just familiar > > > with the Sage review process. I have tested the new functions pretty > > > heavily, and while they are slower in some cases this is dramatically > > > outweighed by the bugfixes. > > > > I think this will be a big advance in the polyhedra functionality in > > > Sage. > > > I see this on the patches: > > > * cddlib-094f.spkg Download (1.8 MB) - added by vbraun 3 months ago. > > cdd_both_reps now reports timing information for individual operations > > * trac_7109_mh1.patch Download (212.8 KB) - added by mhampton 3 weeks ago. > > Based against 4.2.1, cumulative of all changes except cddlib spkg > > > It would be nice if there were a "referee patch" which gets applied > > after vbraun's patch. Then one could see what you did, and what you > > already refereed. As it is, one has to basically start from scratch. > > > -- William -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org