On 4 Jan 2014 22:16, "Dima Pasechnik" <[email protected]> wrote: > > > On 4 Jan 2014 22:11, "Nathann Cohen" <[email protected]> wrote: > > > > Hellooooo !! > > > > > I think you have misunderstood my complaint, as it was about the function > > > returning much more specific thing than asked. You hijack a generic name for > > > something specific. I tried to explain this by an analogy with misnamed > > > NextPrimePower, actually returning NextPrimePower_OF_TWO, which is obviously > > > a better name. > > > > > > One unpleasant side effect of your present design is that it is harder than > > > necessary to extend to cover other projective planes, as you hijack the > > > generic name for no good reason, except your convenience :) > > > > So if I follow you, you agree to have this function named > > > > designs.ProjectivePlaneDesign() for as long as there is an argument named type="Pappian" which does absolutely NOTHING as there is no other implementation available ? > > Sure I do. The argument "Pappian" serves as documentation here, for the time being, it's not absolutely NOTHING... >
Perhaps even better would be to have one more level of classes, i.e. designs.ProjectivePlaneDesign.Pappian() designs.ProjectivePlaneDesign.Hall() etc Even better, drop "Design" part.... > > > > Nathann -- 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 [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/groups/opt_out.
