On Fri, Jul 31, 2015 at 2:16 PM, Nathann Cohen <nathann.co...@gmail.com> wrote: >> I do not have a strong opinion about whether lower case is better or not. >> But I strongly support having a convention and mention it in the developer >> manual. I opened #18974. > > I would say that we already have conventions about upper and lower > cases when it comes to functions (lower) and class names (upper). If > we mix something orthogonal to that, like "name of somebody famous" > there will be collisions. > > Just for fun, I made a list of some names in found in the "schemes/" > folder. I found 30 names there, which appear in lower case in a total > of 416 functions. Of course, you can expect to find much, much more in > Sage. > > If anybody is willing to try, here is my script: > > for name in $(cat case); do grep -R "def.*$name" sage/; done | nl > > All you need is have a 'case' file in your current directory > containing some names (those I have follow this email). > > You will find many, many names. Which ones can you tie to a > mathematical notion? Some are easy (Galois, Legendre, Newton), some > are harder (igusa, tamagawa, kolyvagin). > > I wish good luck to whoever wants to enforce a standard different from > the "mainstream one" (i.e. lower case) :-P
Are any of these names *ever* not lower case in function/method names in the Sage library? If in the _vast_ majority (e.g., 90%+) of the cases they are lower case, then that is in itself a good argument that it should be the official standard, and we should fix the ones that aren't (since clearly that's what people are defaulting to, and they will continue to do so even with a standard). If it's closer to 50/50 or even a majority that are not all lower case, then we should discuss this further. Case insensitive tab completion seems like a good idea. It does feel a bit surprising though, e.g., you type sage: mod[tab] and ModularForms is a "completion", so if you select it, then it has to go back and replace some of what you already typed. William > > Nathann > > P.S.: The list. I added 'petersen' because I know that it occurs in > lower case (e.g. Graph.two_factor_petersen) > > weierstrass > clebsch > igusa > frobenius > teichmuller > coleman > newton > cartier > witt > hasse > cantor > kummer > laurent > monsky_washnitzer > petersen > segre > weil > riemann > kolyvagin > atkin_lehner > galois > bernardi > tamagawa > kodaira > legendre > silverman > cremona > manin > kohel > heegner > > -- > 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/d/optout. -- William (http://wstein.org) -- 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/d/optout.
