Opened 9 years ago

Closed 9 years ago

#16341 closed defect (fixed)

discover_action() fails for unhashable elements

Reported by: saraedum Owned by:
Priority: minor Milestone: sage-6.3
Component: coercion Keywords:
Cc: Merged in:
Authors: Julian Rueth Reviewers: Peter Bruin
Report Upstream: N/A Work issues:
Branch: 4a42be1 (Commits, GitHub, GitLab) Commit: 4a42be10755b8898534f2388e78436f2d714d6dd
Dependencies: Stopgaps:

GitHub link to the corresponding issue

Description (last modified by saraedum)

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'

Change History (5)

comment:1 Changed 9 years ago by saraedum

Branch: u/saraedum/ticket/16341
Created: May 12, 2014, 10:17:29 PMMay 12, 2014, 10:17:29 PM
Modified: May 12, 2014, 10:17:29 PMMay 12, 2014, 10:17:29 PM

comment:2 Changed 9 years ago by saraedum

Authors: Julian Rueth
Description: modified (diff)
Status: newneeds_review

comment:3 Changed 9 years ago by saraedum

Commit: 4a42be10755b8898534f2388e78436f2d714d6dd
Description: modified (diff)

New commits:

4a42be1Fixed discover_action() for unhashable elements

comment:4 Changed 9 years ago by pbruin

Reviewers: Peter Bruin
Status: needs_reviewpositive_review

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

comment:5 Changed 9 years ago by vbraun

Branch: u/saraedum/ticket/163414a42be10755b8898534f2388e78436f2d714d6dd
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.