#16403: Remove Graph.to_partition
-------------------------+-------------------------------------------------
Reporter: | Owner: pdehaye
ncohen | Status: needs_review
Type: | Milestone: sage-6.3
defect | Resolution:
Priority: major | Merged in:
Component: graph | Reviewers:
theory | Work issues:
Keywords: | Commit:
Authors: | 9f08c6b859a0fb853f69dbaedc73811740ee3e91
Nathann Cohen | Stopgaps:
Report Upstream: N/A |
Branch: |
u/ncohen/16403 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by ncohen):
> I do not understand. In the commit you are removing a function, while in
the description you are saying that the function is badly named (but then,
why not rename it?) and that it does not belong there (but you don't
explain why, except pointing to an inconclusive long thread).
>
> Please clarify your grounds for filing a ticket that removes someone
else's work, maybe by editing the description message.
Done. I believe that this function has been added with only FindStat's
interest in mind, not at all Sage's users, which is what Sage is
developped for. I believe that this is sufficient grounds for removing
something.
If you believe that a one-line function like that is useful by itself for
others, and that the authors wanted to implement this for different
reasons than for their own purposes, I am eager to see how you achieve to
explain that while keeping any intellectual honesty.
Please also provide a meaningful name, for the current one is not. I also
personally believe that there is no good name for such a function, which
is also an hint that it may be useless.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/16403#comment:6>
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.
