Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#10309 closed defect (fixed)

Fix doctest error in doc/en/numerical_sage/cvxopt.rst

Reported by: jdemeyer Owned by: mvngu
Priority: major Milestone: sage-4.6.1
Component: doctest coverage Keywords: cvxopt doc
Cc: Merged in: sage-4.6.1.alpha3
Authors: Jeroen Demeyer Reviewers: Dmitrii Pasechnik
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

The error fixed by this patch was introduced by #6456.

Attachments (1)

10309_cvxopt_doctest.patch​ (642 bytes) - added by jdemeyer 9 years ago.

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by jdemeyer

comment:1 Changed 9 years ago by jdemeyer

  • Status changed from new to needs_review

comment:2 follow-up: Changed 9 years ago by dimpase

  • Status changed from needs_review to positive_review

Indeed, yes, this fixes the "fix" introduced in 6456-numerical_sage_cvxopt.patch for reasons I don't see. Positive review.

comment:3 Changed 9 years ago by jdemeyer

  • Reviewers set to Dima Pasechnik

comment:4 Changed 9 years ago by jdemeyer

  • Merged in set to sage-4.6.1.alpha3
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:5 in reply to: ↑ 2 Changed 9 years ago by kcrisman

Replying to dimpase:

Indeed, yes, this fixes the "fix" introduced in 6456-numerical_sage_cvxopt.patch for reasons I don't see. Positive review.

... below since can get -00 or +00 depending on architecture is the relevant explanation, I think.

comment:6 Changed 9 years ago by jdemeyer

  • Reviewers changed from Dima Pasechnik to Dmitrii Pasechnik
Note: See TracTickets for help on using tickets.