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

Error when creating ideal from unhashable elements #13644

Closed
saraedum opened this issue Oct 22, 2012 · 12 comments
Closed

Error when creating ideal from unhashable elements #13644

saraedum opened this issue Oct 22, 2012 · 12 comments

Comments

@saraedum
Copy link
Member

Currently, the following does not work because some padic extension elements do not define a hash function (see #11895 for why they do not define a hash function):

sage: K = Qp(3)
sage: R.<a> = K[]
sage: L.<a> = K.extension(a^2-3)
sage: L.ideal(a)
TypeError: unhashable type: 'sage.rings.padics.padic_ZZ_pX_CR_element.pAdicZZpXCRElement'

The problem is that sage.rings.ring.Ring is looking for a coercion from a to L. There is none, since a is not a parent. The attached patch fixes this.

CC: @roed314

Component: coercion

Keywords: sd59 padics

Author: Julian Rueth

Branch/Commit: 78bb646

Reviewer: Volker Braun

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

@saraedum

This comment has been minimized.

@jdemeyer
Copy link
Contributor

comment:2

Attachment: trac_13644.patch.gz

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@saraedum
Copy link
Member Author

Branch: u/saraedum/ticket/13644

@saraedum
Copy link
Member Author

Changed keywords from none to sd59

@saraedum
Copy link
Member Author

comment:7

CCed roed since this is essentially a ticket about padics.


New commits:

18a8242Trac #13644: fixed a bug in the creation of ideals in padic extension rings

@saraedum
Copy link
Member Author

Changed keywords from sd59 to sd59 padics

@saraedum
Copy link
Member Author

Commit: 18a8242

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 28, 2014

Changed commit from 18a8242 to 78bb646

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 28, 2014

Branch pushed to git repo; I updated commit sha1. New commits:

78bb646fixed format error in docstring

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@vbraun
Copy link
Member

vbraun commented Sep 25, 2014

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Sep 25, 2014

comment:10

lgtm

@vbraun
Copy link
Member

vbraun commented Sep 27, 2014

Changed branch from u/saraedum/ticket/13644 to 78bb646

@vbraun vbraun closed this as completed in c7a4c17 Sep 27, 2014
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