Opened 8 years ago

Closed 8 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:

Status badges

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 8 years ago by saraedum

  • Branch set to u/saraedum/ticket/16341
  • Created changed from 05/12/14 22:17:29 to 05/12/14 22:17:29
  • Modified changed from 05/12/14 22:17:29 to 05/12/14 22:17:29

comment:2 Changed 8 years ago by saraedum

  • Authors set to Julian Rueth
  • Description modified (diff)
  • Status changed from new to needs_review

comment:3 Changed 8 years ago by saraedum

  • Commit set to 4a42be10755b8898534f2388e78436f2d714d6dd
  • Description modified (diff)

New commits:

4a42be1Fixed discover_action() for unhashable elements

comment:4 Changed 8 years ago by pbruin

  • Reviewers set to Peter Bruin
  • Status changed from needs_review to positive_review

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

comment:5 Changed 8 years ago by vbraun

  • Branch changed from u/saraedum/ticket/16341 to 4a42be10755b8898534f2388e78436f2d714d6dd
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.