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

printing unevaluated integrals, limits, etc. broken #7955

Closed
burcin opened this issue Jan 16, 2010 · 6 comments
Closed

printing unevaluated integrals, limits, etc. broken #7955

burcin opened this issue Jan 16, 2010 · 6 comments

Comments

@burcin
Copy link

burcin commented Jan 16, 2010

From the sage-devel thread here:

http://groups.google.com/group/sage-devel/t/592ce36b210c2fbe

On Mon, 11 Jan 2010 23:58:54 -0800 (PST)
"marik@mendelu.cz" <marik@mendelu.cz> wrote:

> Dear sage-devel
> 
> the following (definite) integral is not evaluated by maxima and show
> () command should return the same unevaluated integral in TeX
> notation. I think this was the case in previous versions. On Sage 4.3.
> I get th following
> 
> input: integrate(1/(1+sqrt(x)),x,0,1).show()
> 
> output: \int integrate\,{d \frac{1}{\sqrt{x} + 1}}
> 
> expected output: \int_0^1 \frac{..}{...} dx
> 
> What has changed?

After #7490, we give the function object as the first argument to
custom methods of symbolic functions. The function that prints integrals
is integrate_latex() on line 1556 of sage/calculus/calculus.py. It
gets the function integrate as a first argument, and prints the
nonsense reported above.

Component: symbolics

Author: Burcin Erocal

Reviewer: Dan Drake

Merged: sage-4.3.3.alpha0

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

@burcin burcin added this to the sage-4.3.3 milestone Jan 16, 2010
@burcin burcin self-assigned this Jan 16, 2010
@burcin
Copy link
Author

burcin commented Jan 17, 2010

Attachment: trac_7955-integrate_latex.patch.gz

fix typesetting of unevaluated integrals

@burcin
Copy link
Author

burcin commented Jan 17, 2010

Author: Burcin Erocal

@burcin
Copy link
Author

burcin commented Jan 17, 2010

comment:1

attachment: trac_7955-integrate_latex.patch should fix this.

@dandrake
Copy link
Contributor

comment:2

Replying to @burcin:

attachment: trac_7955-integrate_latex.patch should fix this.

...and it does, on 4.3.1. The code looks good, all doctests pass, and the problem is fixed.

@dandrake
Copy link
Contributor

Reviewer: Dan Drake

@qed777
Copy link
Mannequin

qed777 mannequin commented Feb 11, 2010

Merged: sage-4.3.3.alpha0

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

2 participants