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

DirichletGroup does not work for rings with unhashable elements #16258

Closed
saraedum opened this issue Apr 28, 2014 · 8 comments
Closed

DirichletGroup does not work for rings with unhashable elements #16258

saraedum opened this issue Apr 28, 2014 · 8 comments

Comments

@saraedum
Copy link
Member

The following fails because p-adics are not hashable:

sage: K.<a> = Qq(9)
sage: DirichletGroup(2,base_ring=K,zeta=K(-1),zeta_order=2)
TypeError: unhashable type: 'sage.rings.padics.padic_ZZ_pX_CR_element.pAdicZZpXCRElement'

Depends on #16317

Component: modular forms

Keywords: sd59, hash

Author: Julian Rueth

Branch/Commit: u/saraedum/ticket/16258 @ 6ce4fc8

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

@saraedum saraedum added this to the sage-6.2 milestone Apr 28, 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

saraedum commented May 9, 2014

Branch: u/saraedum/ticket/16258

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 26, 2014

Commit: 6ce4fc8

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 26, 2014

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

78879f0Merge branch 'develop' into ticket/16258
7a0f094Introduced _cache_key for sage objects
fa16bc7Merge branch 'u/saraedum/ticket/16251' of git://trac.sagemath.org/sage into ticket/16316
798aaf8Implemented _cache_key() for polynomials
877302eEnable caching for non-hashable objects
ba9befaEnabled caching for unhashable objects in factories
f661642Merge branch 'u/saraedum/ticket/16317' of git://trac.sagemath.org/sage into ticket/16258
6ce4fc8fixed doctest

@saraedum
Copy link
Member Author

Dependencies: #16317

@saraedum
Copy link
Member Author

Author: Julian Rueth

@saraedum
Copy link
Member Author

Changed keywords from none to sd59, hash

@jdemeyer
Copy link

jdemeyer commented May 6, 2015

comment:7

What's the relation between #16342 and #16258? In any case, needs rebasing.

@saraedum
Copy link
Member Author

comment:8

Oops, it seems I fixed the same bug twice.

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