Opened 8 years ago

Closed 8 years ago

#12511 closed defect (fixed)

Unable to take numeric powers of I

Reported by: robertwb Owned by: tbd
Priority: blocker Milestone: sage-5.0
Component: symbolics Keywords:
Cc: was Merged in: sage-5.0.beta5
Authors: Robert Bradshaw Reviewers: Burcin Erocal
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by kini)

sage: I ^ 0.5
None

Apply 12511-I-pow.fixed.patch to $SAGE_ROOT/devel/sage-main

Attachments (2)

12511-I-pow.patch (3.5 KB) - added by robertwb 8 years ago.
12511-I-pow.fixed.patch (3.3 KB) - added by jdemeyer 8 years ago.
Rebased to sage-5.0.beta4, apply only this

Download all attachments as: .zip

Change History (11)

Changed 8 years ago by robertwb

comment:1 Changed 8 years ago by robertwb

  • Status changed from new to needs_review

comment:2 Changed 8 years ago by kini

  • Component changed from PLEASE CHANGE to symbolics

comment:3 Changed 8 years ago by robertwb

FYI, I changed "return None" to raise an error to avoid (and make more debuggable) future issues of this sort. For places where None is explicitly acceptable (in the bowels of Pynac) it can be returned explicitly in the except clause.

comment:4 Changed 8 years ago by aapitzsch

Use the Python 3 compatible syntax to raise an error

raise ValueError("recursion")

comment:5 Changed 8 years ago by burcin

  • Reviewers set to Burcin Erocal
  • Status changed from needs_review to positive_review

Thanks for the quick fix!

comment:6 follow-up: Changed 8 years ago by kini

  • Description modified (diff)

Burcin, I trust this fix doesn't cancel your positive review?

comment:7 in reply to: ↑ 6 ; follow-up: Changed 8 years ago by burcin

Replying to kini:

Burcin, I trust this fix doesn't cancel your positive review?

Good catch! Still positive review. :)

comment:8 in reply to: ↑ 7 Changed 8 years ago by kini

Replying to burcin:

Good catch!

aapitzsch is the one who caught it, I just fixed it :)

Changed 8 years ago by jdemeyer

Rebased to sage-5.0.beta4, apply only this

comment:9 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.0.beta5
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.