On 2013-06-19, Nathann Cohen <nathann.co...@gmail.com> wrote:
> --20cf301ee7a5191d5204df850b33
> Content-Type: text/plain; charset=ISO-8859-1
>
> Hellooooooooo everybody !
>
> I just noticed in today's release the presence of two tickets that I had
> missed while they were being reviewed, and I really regret it.
>
> Those two tickets are #14734 and #14732. Each of them adds to Graph and
> Poset a one-line method which I personnally find totally useless. Of
> course, while I don't see the point of these methods, I don't expect to
> find everything that Sage can do meaningful, as I have after all my own
> personnal restricted field of interest, *but* the fact that these functions
> could be written in 30 characters really makes me think harder.

Deliberately ignoring the decorator technicalities, I think both #14734
and #14732 have been wrongly designed. (Throw rotten tomatoes at me
now).

Regarding #14734: well, if one adds to Graph a grow_bushy_tail() method
indeed this goes too far. I personally don't get why Bushy_Tail cannot
get a constructor from Graph instead.  This would only be justified if
growing bushy tails inside Graph is more efficient than doing it outside
of it.

Same applies to #14732: why on Earth it makes Posets vomit ((c) V.Braun)
Graphs all of a sudden? Shouldn't rather Graph get a constructor from
Posets?

Having said that, the gals and guys responsible for #14734 and #14732 are at
Sagedays 49 now, go and buy them the necessary amount of beer to have
all this sorted out :-)

Dima

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to