Ticket #709 (closed enhancement: fixed)

Opened 6 years ago

Last modified 6 years ago

[with patch] Add doctests to ensure that scipy and cvxopt build correctly on all platforms.

Reported by: jkantor Owned by: jkantor
Priority: major Milestone: sage-2.8.8
Component: packages: standard Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Scipy and cvxopt tend to appear to build correctly, but then raise exceptions when modules are imported (usually missing symbols). We need doctests so that this is detected when tests are run.

Attachments

709_bugfix.hg Download (603 bytes) - added by jkantor 6 years ago.

Change History

Changed 6 years ago by jkantor

comment:1 Changed 6 years ago by jkantor

The above patch adds a file that is doctested and tests importing modules from cvxopt and scipy that are known to have problems. This together with the fix for bug 700 should fix the cvxopt problem as well as detect future breakage which is good since cvxopt silently broke when we switched from gfortran to g95.

comment:2 Changed 6 years ago by was

  • Status changed from new to closed
  • Resolution set to fixed

comment:3 Changed 6 years ago by was

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Milestone changed from sage-2.8.4.3 to sage-2.9

I've included this but put a nodoctest in the file until trac #700 is resolved.

comment:4 Changed 6 years ago by mhansen

  • Summary changed from Add doctests to ensure that scipy and cvxopt build correctly on all platforms. to [with patch] Add doctests to ensure that scipy and cvxopt build correctly on all platforms.

comment:5 Changed 6 years ago by mabshoff

  • Milestone changed from sage-2.9 to sage-2.8.8

comment:6 Changed 6 years ago by was

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.