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

Expression._latex_() triggers “dangerous” comparisons #22071

Closed
mezzarobba opened this issue Dec 16, 2016 · 2 comments
Closed

Expression._latex_() triggers “dangerous” comparisons #22071

mezzarobba opened this issue Dec 16, 2016 · 2 comments

Comments

@mezzarobba
Copy link
Member

With commit a5a1e2b76332d136936c7cbe94cf42f93ec6203d from ticket #22029:

sage: Mod(2,7)/x
2/x
sage: latex(Mod(2,7)/x)
...
TypeError: unsupported operand parent(s) for 'lt': 'Ring of integers modulo 7' and 'Number Field in I with defining polynomial x^2 + 1'

It looks like Pynac is comparing the numerator with I under the hood. Even if the branch in question isn't merged, this is likely to cause problems sooner or later.

CC: @rwst

Component: symbolics

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

@mezzarobba mezzarobba added this to the sage-7.5 milestone Dec 16, 2016
@rwst
Copy link

rwst commented Dec 17, 2016

comment:1

Note there is a consensus that operations with positive characteristic ring elements and symbolic variables should be discouraged (leading to #21391). Consequently the latexing will fail as well as soon as that is implemented, and you would need something like #10035 to make it work. That doesn't fix this specific odd error but it lowers the urgency level I'm personally having with it.

@rwst
Copy link

rwst commented Jun 7, 2017

comment:2

Seems to be no longer an issue.

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