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.

Sign up for GitHub

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

Jump to bottom

normalization in Pynac buggy #19775

Closed
rwst opened this issue Dec 24, 2015 · 10 comments
Closed

normalization in Pynac buggy #19775

rwst opened this issue Dec 24, 2015 · 10 comments
Labels
c: symbolics r: duplicate t: bug
Milestone
sage-7.3

Comments

@rwst
Copy link

rwst commented Dec 24, 2015

from https://groups.google.com/forum/?hl=en#!topic/sage-devel/awa0lNGJugo
Minimal version:

sage: a,b,c,d,e=var('a,b,c,d,e')
sage: (c/a - b*c^2/(a^2*(b*c/a-d)) + c*d/(a*(b*c/a-d))).normalize()
0
sage: (e + c/a - b*c^2/(a^2*(b*c/a-d)) + c*d/(a*(b*c/a-d))).normalize()
(a^2*b*c*e - a^3*d*e)/((b*c - a*d)*a^2)

This changed somewhere between 6.6 (output e) and 7.0.

See pynac/pynac#111

Depends on #21034

Upstream: Fixed upstream, in a later stable release.

Component: symbolics

Author: Ralf Stephan

Reviewer: Paul Masson

Issue created by migration from https://trac.sagemath.org/ticket/19775
@rwst rwst added this to the sage-7.0 milestone Dec 24, 2015
@rwst
Copy link
Author

rwst commented Dec 24, 2015

Upstream: Reported upstream. Developers acknowledge bug.

@rwst

This comment has been minimized.

Sign in to view
@rwst

This comment has been minimized.

Sign in to view
@rwst

This comment has been minimized.

Sign in to view
@rwst
Copy link
Author

rwst commented Jun 18, 2016

comment:4

Fixed in pynac-0.6.8.

@rwst
Copy link
Author

rwst commented Jun 18, 2016

Changed upstream from Reported upstream. Developers acknowledge bug. to Fixed upstream, in a later stable release.

@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Jul 23, 2016

Dependencies: #21034

@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Jul 23, 2016

Reviewer: Paul Masson

@paulmasson paulmasson mannequin added the s: needs review label Jul 23, 2016
@paulmasson paulmasson mannequin modified the milestones: sage-7.0, sage-7.3 Jul 23, 2016
@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Jul 27, 2016

comment:6

Fixed in #21034

@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Jul 27, 2016

Author: Ralf Stephan

@vbraun vbraun closed this as completed Aug 15, 2016
@rwst rwst mentioned this issue Aug 7, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: symbolics r: duplicate t: bug
Projects
None yet
Milestone
sage-7.3
Development

No branches or pull requests
2 participants
@vbraun @rwst