#16553: Clean IncidenceStructure
-------------------------------------+-------------------------------------
Reporter: vdelecroix | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: combinatorial | Resolution:
designs | Merged in:
Keywords: | Reviewers:
Authors: Nathann Cohen, | Work issues:
Vincent Delecroix | Commit:
Report Upstream: N/A | b51512d642a603ea91d9358ea3d7877f184938d0
Branch: public/16553 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by vdelecroix):
Replying to [comment:59 ncohen]:
> > I would like it for sure if its purpose was only to return a boolean!
A function that starts with `is` must return a boolean, otherwise it is a
mess:
> > {{{
> > if m.is_even():
> > print "m is even"
> > }}}
> > bad luck: `.is_even()` return a pair "(boolean, m divided by 2)"
(remember the concrete example in #16464).
>
> `return_parameters=True`...
>
> > It is implemented, not very ugly and not very useful!
>
> What about all the stuff you converted to tuples ?..
I do not understand your point, in [comment:58] you proposed
`is_t_design(self,t,v,k,l)`. Now, you want to keep the
`return_parameters=False/True`. Fine, but it will be ugly if you make only
one function.
If you want to propose some code, do it! I am lost!
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/16553#comment:60>
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.
