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

discover_action() fails for unhashable elements #16341

Closed
saraedum opened this issue May 12, 2014 · 9 comments
Closed

discover_action() fails for unhashable elements #16341

saraedum opened this issue May 12, 2014 · 9 comments
Labels
c: coercion p: minor / 4 t: bug
Milestone
sage-6.3

Comments

@saraedum
Copy link
Member

The following currently fails::

sage: K.<a> = Qq(9)
sage: E=EllipticCurve_from_j(0).base_extend(K)
sage: E.get_action(ZZ)
TypeError: unhashable type: 'sage.rings.padics.padic_ZZ_pX_CR_element.pAdicZZpXCRElement'

Component: coercion

Author: Julian Rueth

Branch/Commit: 4a42be1

Reviewer: Peter Bruin

Issue created by migration from https://trac.sagemath.org/ticket/16341
@saraedum saraedum added this to the sage-6.3 milestone May 12, 2014
@saraedum
Copy link
Member Author

Branch: u/saraedum/ticket/16341

@saraedum

This comment has been minimized.

Sign in to view
@saraedum
Copy link
Member Author

Author: Julian Rueth

@saraedum

This comment has been minimized.

Sign in to view
@saraedum
Copy link
Member Author

Commit: 4a42be1

@saraedum
Copy link
Member Author

New commits:

4a42be1Fixed discover_action() for unhashable elements

@pjbruin
Copy link
Contributor

pjbruin commented May 22, 2014

Reviewer: Peter Bruin

@pjbruin
Copy link
Contributor

pjbruin commented May 22, 2014

comment:4

Looks good to me, given that comparison is identity-based, and the patchbot is happy too.

@vbraun
Copy link
Member

vbraun commented May 23, 2014

Changed branch from u/saraedum/ticket/16341 to 4a42be1

@vbraun vbraun closed this as completed in 75952e0 May 23, 2014
@mmasdeu mmasdeu mentioned this issue Mar 26, 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: coercion p: minor / 4 t: bug
Projects
None yet
Milestone
sage-6.3
Development

No branches or pull requests
3 participants
@saraedum @vbraun @pjbruin