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

fractional powers for polynomial variables bug #3678

Closed
wdjoyner opened this issue Jul 19, 2008 · 5 comments
Closed

fractional powers for polynomial variables bug #3678

wdjoyner opened this issue Jul 19, 2008 · 5 comments

Comments

@wdjoyner
Copy link

Raising a multivariate polynomial to a fractional power is broken.
A TypeError should be raised, but it is rounded instead.

sage: pr = PolynomialRing(QQ, "u,v")
sage: pr.injvar()
Defining u, v
sage: u^(1/2)
1

Component: algebra

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

@wdjoyner wdjoyner added this to the sage-3.4 milestone Jul 19, 2008
@williamstein

This comment has been minimized.

@williamstein williamstein changed the title Fractional powers for polynomial variables bug fractional powers for polynomial variables bug Jul 19, 2008
@aghitza
Copy link

aghitza commented Dec 21, 2008

comment:2

The attached patch fixes this by introducing the same type-check as in the univariate polynomial case.

@aghitza
Copy link

aghitza commented Dec 21, 2008

Attachment: trac_3678.patch.gz

@JohnCremona
Copy link
Member

comment:3

Positive review. Patch applies cleanly to 3.2.2 and tests in sage/rings/polynomial pass.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Dec 21, 2008

comment:4

Merged in Sage 3.2.3.alpha0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Dec 21, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.4, sage-3.2.3 Dec 21, 2008
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