Ask Your Question
1

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

asked 2014-01-17 16:38:00 +0100

anonymous user

Anonymous

updated 2023-01-10 00:01:09 +0100

tmonteil 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 08:53:16 +0100

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 16:22:06 +0100 )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 16:38:00 +0100

Seen: 683 times

Last updated: Jan 18 '14