Opened 5 years ago

Closed 5 years ago

#18297 closed defect (fixed)

Increase tolerance in a doctest in linear programming tutorial

Reported by: vdelecroix Owned by:
Priority: major Milestone: sage-6.7
Component: documentation Keywords:
Cc: ncohen Merged in:
Authors: Vincent Delecroix Reviewers: Nathann Cohen
Report Upstream: N/A Work issues:
Branch: 6494c99 (Commits) Commit: 6494c992ba24981bfaf47d3b83df145d03fd81d5
Dependencies: Stopgaps:

Description

If one has Coin-or installed (and a self compiled version of Sage) then a doctest fails in the linear programming tutorial

sage -t linear_programming.rst
**********************************************************************
File "linear_programming.rst", line 432, in doc.en.thematic_tutorials.linear_programming
Failed example:
    p.solve()  # rel tol 2e-12
Expected:
    2.0
Got:
    2.0000000000040004
Tolerance exceeded:
    2.0 vs 2.0000000000040004, tolerance 2e-12 > 2e-12
**********************************************************************

We just increase the tolerance to 2e-11.

See also this sage-devel thread.

Change History (5)

comment:1 Changed 5 years ago by vdelecroix

  • Branch set to u/vdelecroix/18297
  • Commit set to 6494c992ba24981bfaf47d3b83df145d03fd81d5
  • Status changed from new to needs_review

New commits:

6494c99Trac 18297: increase doctest tolerance

comment:2 follow-up: Changed 5 years ago by ncohen

Was it the the only broken doctest ? O_o

Last edited 5 years ago by ncohen (previous) (diff)

comment:3 in reply to: ↑ 2 Changed 5 years ago by vdelecroix

Replying to ncohen:

Was it the the only broken doctest ? O_o

Yes. At least for me: patchbot shortlog

Vincent

comment:4 Changed 5 years ago by ncohen

  • Reviewers set to Nathann Cohen
  • Status changed from needs_review to positive_review

Oh. You have cbc installed on a patchbot. I see :-D

Nathann

comment:5 Changed 5 years ago by vbraun

  • Branch changed from u/vdelecroix/18297 to 6494c992ba24981bfaf47d3b83df145d03fd81d5
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.