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

Bug in latex for power series with greek variable names #26606

Closed
roed314 opened this issue Oct 31, 2018 · 16 comments
Closed

Bug in latex for power series with greek variable names #26606

roed314 opened this issue Oct 31, 2018 · 16 comments

Comments

@roed314
Copy link
Contributor

roed314 commented Oct 31, 2018

sage: R.<beta> = QQ[]
sage: S.<x> = R[[]]
sage: latex(beta*x)
\betax

See also:

Depends on #29892

CC: @slel

Component: algebra

Author: Hoa Nguyen

Branch/Commit: c98e706

Reviewer: Frédéric Chapoton

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

@roed314 roed314 added this to the sage-8.5 milestone Oct 31, 2018
@DaveWitteMorris
Copy link
Member

comment:1

The bug will probably be easy to fix if its location in the source code can be found. Presumably, the latex string for a power series is constructed by concatenating the latex strings of its parts. To kill the bug, add a space in between: change latex(a) + latex(b) to latex(a) + ' ' + latex(b). This is safe, because latex ignores extraneous spaces in math formulas.

(For anyone who does not know latex: \betax should be \beta x.)

@mkoeppe
Copy link
Contributor

mkoeppe commented Jun 18, 2020

Dependencies: #29869

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 5, 2020

Changed dependencies from #29869 to #29892

@hnpl
Copy link
Member

hnpl commented Oct 18, 2020

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 18, 2020

Branch pushed to git repo; I updated commit sha1. New commits:

dd4e4d3Add a space between x and var in PowerSeries._latex_()

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 18, 2020

Commit: dd4e4d3

@fchapoton
Copy link
Contributor

comment:8

Hello, please

  • add your full real name here above in the Authors: field of the trac page

  • add a doctest for your fix in the modified method documentation

  • set the status to "needs_review" once done

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 18, 2020

Changed commit from dd4e4d3 to 2823ab4

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 18, 2020

Branch pushed to git repo; I updated commit sha1. New commits:

2823ab4Add doctest

@hnpl
Copy link
Member

hnpl commented Oct 18, 2020

Author: Hoa Nguyen

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 19, 2020

Branch pushed to git repo; I updated commit sha1. New commits:

534e473Removing trailing whitespaces on the left using str.lstrip(" ")
c98e706Modify previous doctest of PowerSeries._latex_()

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 19, 2020

Changed commit from 2823ab4 to c98e706

@fchapoton
Copy link
Contributor

comment:12

ok, thanks. Looks good

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@vbraun
Copy link
Member

vbraun commented Nov 7, 2020

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

6 participants