integer_nthroot and factorint -> iroot and ifactor

5 views
Skip to first unread message

smichr

unread,
May 27, 2011, 1:59:18 AM5/27/11
to sympy
What do you think about changing the names integer_nthroot and
factorint -> iroot and ifactor?

Aaron S. Meurer

unread,
May 27, 2011, 2:39:35 AM5/27/11
to sy...@googlegroups.com
I don't think it's worth it.

Aaron Meurer

Chris Smith

unread,
May 27, 2011, 2:43:24 AM5/27/11
to sy...@googlegroups.com
I think that with igcd, ilcm, igcdex, ifib, and ifac there is precedence for using these simple names. Tooltips are going to clue someone in about this functionality, too.

Vinzent Steinberg

unread,
May 27, 2011, 10:44:51 AM5/27/11
to sympy
On 27 Mai, 08:43, Chris Smith <smi...@gmail.com> wrote:
> I think that with igcd, ilcm, igcdex, ifib, and ifac there is precedence for
> using these simple names.

This is a good point. However the less verbose names are cryptic to
me, whereas the current ones are easy to understand intuitively. What
about having both names?

Vinzent

Chris Smith

unread,
May 27, 2011, 10:57:56 AM5/27/11
to sy...@googlegroups.com
That's ok with me. Let's see if anyone else has an opinion.

Ondrej Certik

unread,
May 27, 2011, 1:56:50 PM5/27/11
to sy...@googlegroups.com
On Fri, May 27, 2011 at 7:57 AM, Chris Smith <smi...@gmail.com> wrote:
> That's ok with me. Let's see if anyone else has an opinion.

I for myself prefer the longer names. I don't think this is used so
often, so that you should shorten it. But I am ok either way.

Ondrej

Reply all
Reply to author
Forward
0 new messages