#17979: Reimplementation of IntegerListsLex
-------------------------------------+-------------------------------------
Reporter: aschilling | Owner:
Type: defect | Status: needs_review
Priority: blocker | Milestone: sage-6.6
Component: combinatorics | Resolution:
Keywords: days64 | Merged in:
Authors: Bryan Gillespie, | Reviewers: Nathann Cohen, Jeroen
Anne Schilling, Nicolas M. Thiery | Demeyer, Travis Scrimshaw
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/ticket/17979 | 2c7019c64047f5f84d89d72fec81a7af483cdc66
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by ncohen):
> We have been working hard to make the deadline for
> the release. Like in any piece of code, there certainly are other
> internal micro-refactoring that could possibly be done. If you note
> one, please be specific on whether you believe it really needs to be
> done now, or could be postponed to one of the follow up tickets;
> especially since some of those tickets may well induce complete
> rewrite of the ditto micro-refactored code anyway.
Have you had the impression that some of our previous requests were
abusive? We try to understand your code, make sure that it is correct
maintenable and thus that it will be easy to understand and work on it in
the future.
Many of the things that seem obvious to the author are not easily deduced
when you only look at the code, and unexpected design choices often raise
more questions.
I personally gave up raising some arguments (that still seem legitimate to
me) only to lessen the load on this ticket. I still plan to read the code
you implement totally (I haven't done that yet), and I will probably have
more time to do so next week.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/17979#comment:385>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
