Opened 11 years ago

Closed 11 years ago

#6998 closed defect (fixed)

[with patch, positive review] wrong error for powerseries sqrt

Reported by: robertwb Owned by: mhansen
Priority: minor Milestone: sage-4.2
Component: algebra Keywords:
Cc: Merged in: sage-4.2.alpha0
Authors: Mike Hansen Reviewers: Adam Webb
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

sage: R.<x> = QQ[[]]
sage: (x^10/2).sqrt()
Traceback (click to the left for traceback)
...
ValueError: power series does not have a square root since it has odd
valuation.

Attachments (1)

trac_6998.patch (3.9 KB) - added by mhansen 11 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 11 years ago by robertwb

  • Component changed from basic arithmetic to algebra
  • Owner changed from somebody to tbd

Changed 11 years ago by mhansen

comment:2 Changed 11 years ago by mhansen

  • Authors set to Mike Hansen
  • Owner changed from tbd to mhansen
  • Status changed from new to assigned
  • Summary changed from wrong error for powerseries sqrt to [with patch, needs review] wrong error for powerseries sqrt

comment:3 Changed 11 years ago by awebb

  • Summary changed from [with patch, needs review] wrong error for powerseries sqrt to [with patch, positive review] wrong error for powerseries sqrt

The patch works as advertised so I give it a positive review. The doc also builds correctly.

Adam

comment:4 Changed 11 years ago by mhansen

  • Merged in set to sage-4.2.alpha0
  • Resolution set to fixed
  • Reviewers set to Adam Webb
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.