Steven D'Aprano <steve+pyt...@pearwood.info> added the comment:
> This involved a few changes, which seem to reflect the consensus here: > - raise ValueError if k>n ; > - rename the function to math.combinations. I see at least four people (myself, Raymond, Mark and Tim) giving comb as first choice, and I didn't see anyone give combinations as their first choice. I don't object to you taking it upon yourself to go with the longer name (which is my second choice), but I do object to you claiming concensus for the change without evidence of such. > There was also no reply to my comment about `comb` being confusing > (due to the collision with an English word). To be honest, I didn't think that comment needed a reply. Collisions between words in different knowledge domains are not unusual. I don't think people think that math.tan has anything to do with changing the colour of their skin, or math.sin is about being wicked. Due to their length, permutation, combination and factorial are frequently abbreviated to perm, comb, fact and anyone looking for those functions should recognise the abbreviations. But given the precedent set by itertools and math.factorial, perhaps you are right and we ought to stick to the longer name. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue35431> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
