Opened 3 years ago

Closed 3 years ago

#25252 closed defect (fixed)

Doctest: Complex arithmetic/exponentiation hang (or very slow)

Reported by: gh-aerusso Owned by:
Priority: major Milestone: sage-8.3
Component: algebra Keywords: days94
Cc: rws Merged in:
Authors: Ralf Stephan Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 8b25b1c (Commits, GitHub, GitLab) Commit: 8b25b1c5f7e1ec2587afc752d98b4beefea16f64
Dependencies: #24838, #25251 Stopgaps:

Status badges

Description

Debian unstable (either downloaded directly or from the Debian repositories), running 8.1. Trying to multiply

(1+exp(i*pi/4))*exp(i*pi/4)

does not terminate (in a reasonable amount of time)

exp(i*pi/4)+exp(i*pi/4)*exp(i*pi/4)

on the other hand, resolves fine.

Sorry if this is a duplicate bug.

Change History (8)

comment:1 Changed 3 years ago by egourgoulhon

It's OK in Sage 8.1.beta5 and all previous versions of Sage:

sage: (1+exp(i*pi/4))*exp(i*pi/4) 
-(1/4*I + 1/4)*sqrt(2)*(-(I + 1)*sqrt(2) - 2)

It hangs in Sage 8.1.beta8. So the bug was introduced between Sage 8.1.beta6 and 8.1.beta8.

comment:2 Changed 3 years ago by egourgoulhon

  • Cc rws added

comment:3 Changed 3 years ago by rws

  • Dependencies set to #24838

Fixed in pynac-0.7.17 (#24838). See also #24768. This ticket should add a doctest.

comment:4 Changed 3 years ago by rws

  • Branch set to u/rws/25252

comment:5 Changed 3 years ago by rws

  • Authors set to Ralf Stephan
  • Commit set to ee8bff4575686c5024b889884e1d5a2de5b91b46
  • Dependencies changed from #24838 to #24838, #25251
  • Milestone changed from sage-8.2 to sage-8.3
  • Status changed from new to needs_review
  • Summary changed from Complex arithmetic/exponentiation hang (or very slow) to Doctest: Complex arithmetic/exponentiation hang (or very slow)

New commits:

495e9f425251: doctest
ee8bff425252: doctest25252: doctest25252: doctest25252: doctest25252:

comment:6 Changed 3 years ago by git

  • Commit changed from ee8bff4575686c5024b889884e1d5a2de5b91b46 to 8b25b1c5f7e1ec2587afc752d98b4beefea16f64

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

8b25b1c25252: doctest

comment:7 Changed 3 years ago by tscrim

  • Keywords days94 added
  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

LGTM.

comment:8 Changed 3 years ago by vbraun

  • Branch changed from u/rws/25252 to 8b25b1c5f7e1ec2587afc752d98b4beefea16f64
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.