Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

System-dependent term order for printed expressions #9632

Closed
qed777 mannequin opened this issue Jul 29, 2010 · 8 comments
Closed

System-dependent term order for printed expressions #9632

qed777 mannequin opened this issue Jul 29, 2010 · 8 comments

Comments

@qed777
Copy link
Mannequin

qed777 mannequin commented Jul 29, 2010

The order in which the terms in some symbolic expressions are printed depends on the platform/system. For example, evaluating cos(x) + zeta(x) yields

  • zeta(x) + cos(x) on Linux
  • cos(x) + zeta(x) on OS X

in Sage 4.4.4 and 4.5.2.alpha{0,1}, at least.

Please see #9582 for some details and discussion.

Apply: attachment: trac_9632-doctests.patch

CC: @burcin @sagetrac-cwitty @dandrake @jhpalmieri @kcrisman

Component: symbolics

Author: Burcin Erocal

Reviewer: Volker Braun

Merged: sage-5.11.rc0

Issue created by migration from https://trac.sagemath.org/ticket/9632

@qed777 qed777 mannequin added c: symbolics labels Jul 29, 2010
@qed777 qed777 mannequin assigned burcin Jul 29, 2010
@burcin
Copy link

burcin commented Nov 18, 2010

comment:1

#10282 is a duplicate of this issue.

The patches attached to #9880 fix this.

@burcin burcin added this to the sage-4.6.1 milestone Nov 18, 2010
@kcrisman
Copy link
Member

comment:2

#10282 almost certainly is the same thing.

@kcrisman
Copy link
Member

comment:3

When this is closed (which would hopefully happen with #9880 integrated), let's be sure to write doctests for both this ticket and #10282, which would just be to say that

sage: psi(1,1/3)*log(3)
log(3)*psi(1, 1/3)

is the same on all systems, in addition to the zeta(x)+cos(x) example here.

@burcin
Copy link

burcin commented Jul 1, 2013

Author: Burcin Erocal

@burcin
Copy link

burcin commented Jul 1, 2013

comment:4

Attachment: trac_9632-doctests.patch.gz

This was fixed by #9880. attachment: trac_9632-doctests.patch adds doctests.

@burcin

This comment has been minimized.

@vbraun
Copy link
Member

vbraun commented Jul 12, 2013

Reviewer: Volker Braun

@jdemeyer
Copy link

Merged: sage-5.11.rc0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants