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

Stopgap for factorization over fraction fields #23643

Closed
saraedum opened this issue Aug 18, 2017 · 11 comments
Closed

Stopgap for factorization over fraction fields #23643

saraedum opened this issue Aug 18, 2017 · 11 comments

Comments

@saraedum
Copy link
Member

Stopgap warning for #23642.

Component: commutative algebra

Reviewer: Julian Rüth

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

@saraedum
Copy link
Member Author

@saraedum
Copy link
Member Author

Changed branch from u/saraedum/stopgap_for_factorization_over_fraction_fields to none

@saraedum
Copy link
Member Author

comment:2

Now it prints:

sage: sage: k.<a> = GF(9)
....: sage: K = k['t'].fraction_field()
....: sage: R.<x> = K[]
....: sage: f = x^3 + a
....: sage: f.factor()
....: 
/home/jule/proj/sage/sage/local/lib/python2.7/site-packages/IPython/core/interactiveshell.py:2881:
********************************************************************************
Factorization over some quotients can be incorrect.
This issue is being tracked at https://github.com/sagemath/sage-prod/issues/23642.
********************************************************************************
x^3 + a

@saraedum
Copy link
Member Author

Commit: 4b88524

@saraedum
Copy link
Member Author

@saraedum
Copy link
Member Author

New commits:

4b88524Stopgap warning for 23642

@roed314
Copy link
Contributor

roed314 commented Oct 4, 2017

comment:4

Since #23642 has positive review, I think this can be closed as won't fix.

@roed314
Copy link
Contributor

roed314 commented Oct 4, 2017

Changed branch from u/saraedum/stopgap_for_factorization_over_fraction_fields to none

@roed314
Copy link
Contributor

roed314 commented Oct 4, 2017

Changed commit from 4b88524 to none

@roed314 roed314 removed this from the sage-8.1 milestone Oct 4, 2017
@jdemeyer
Copy link

Changed author from Julian Rüth to none

@jdemeyer
Copy link

Reviewer: Julian Rüth

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

3 participants