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

Content of general polynomial not defined. #12218

Closed
sagetrac-mkosters mannequin opened this issue Dec 21, 2011 · 9 comments
Closed

Content of general polynomial not defined. #12218

sagetrac-mkosters mannequin opened this issue Dec 21, 2011 · 9 comments
Labels
c: algebra p: major / 3 t: enhancement
Milestone
sage-5.0

Comments

@sagetrac-mkosters
Copy link
Mannequin

sagetrac-mkosters mannequin commented Dec 21, 2011

The content of a general polynomial (which is the ideal generated by its coefficients) is not defined.

Component: algebra

Keywords: sd35

Author: Michiel Kosters

Reviewer: David Loeffler

Merged: sage-5.0.beta0

Issue created by migration from https://trac.sagemath.org/ticket/12218
@sagetrac-mkosters sagetrac-mkosters mannequin added this to the sage-4.8 milestone Dec 21, 2011
@sagetrac-mkosters
Copy link
Mannequin Author

sagetrac-mkosters mannequin commented Dec 21, 2011

comment:1

Attachment: 12218.patch.gz

@sagetrac-mkosters

This comment has been minimized.

Sign in to view
@saraedum
Copy link
Member

comment:3

Your patch is missing an email address. Probably the system you created the patch on was not configured correctly; see http://www.sagemath.org/doc/developer/walk_through.html#submitting-a-change

@sagetrac-mkosters
Copy link
Mannequin Author

sagetrac-mkosters mannequin commented Dec 21, 2011

Now with my email address

@sagetrac-mkosters
Copy link
Mannequin Author

sagetrac-mkosters mannequin commented Dec 21, 2011

comment:4

Attachment: 12218_2.patch.gz

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Dec 25, 2011

comment:5

Looks good to me. It is absurd that Sage doesn't recognize that the ideal {{{(2, 3, 1)}} of the integers mod 4 is the whole ring, but that is nothing to do with this patch.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Dec 25, 2011

Reviewer: David Loeffler

@jdemeyer jdemeyer modified the milestones: sage-4.8, sage-5.0 Dec 26, 2011
@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jan 5, 2012

Changed keywords from none to sd35

@jdemeyer
Copy link
Contributor

Merged: sage-5.0.beta0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: algebra p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-5.0
Development

No branches or pull requests
2 participants
@saraedum @jdemeyer