Ask Your Question
1

simplify_full - is the mistake in the documentation or the source code?

asked 2014-01-17 09:38:00 -0500

anonymous user

Anonymous

updated 2015-01-18 01:43:59 -0500

FrédéricC gravatar image

Looking at the simplify_full documentation in Sage 6.0, it says that it

Applies simplify_factorial, simplify_trig, simplify_rational, simplify_radical, simplify_log, and again simplify_rational to self (in that order).

However the source code is

def simplify_full(self):
    x = self
    x = x.simplify_factorial()
    x = x.simplify_trig()
    x = x.simplify_rational()
    x = x.simplify_log('one')
    x = x.simplify_rational()
    return x

So where is the mistake? Should simplify_radical be added to the source code or removed from the documentation?

edit retag flag offensive close merge delete

1 answer

Sort by » oldest newest most voted
2

answered 2014-01-18 01:53:16 -0500

tmonteil gravatar image

Good catch ! simplify_radical() was removed from simplify_full() and full_simplify() in trac ticket 12737, but not from the documentation. I created trac ticket 15693 to solve this.

Thanks for reporting.

edit flag offensive delete link more

Comments

There is a fix up as well, so hopefully this will be done soon.

kcrisman gravatar imagekcrisman ( 2014-01-23 09:22:06 -0500 )edit

Your Answer

Please start posting anonymously - your entry will be published after you log in or create a new account.

Add Answer

Question Tools

Stats

Asked: 2014-01-17 09:38:00 -0500

Seen: 312 times

Last updated: Jan 18 '14