Opened 6 years ago

Closed 6 years ago

#21499 closed enhancement (fixed)

GenericBackend: Add _test_sense method

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-7.5
Component: numerical Keywords: lp
Cc: dimpase Merged in:
Authors: Matthias Koeppe Reviewers: Dima Pasechnik
Report Upstream: N/A Work issues:
Branch: a3a9a0f (Commits, GitHub, GitLab) Commit: a3a9a0f6cf826d27b2554e831133ddf2f303350d
Dependencies: Stopgaps:

Status badges


This adds another test method, which all MIP backends pass.

Change History (7)

comment:1 Changed 6 years ago by mkoeppe

  • Branch set to u/mkoeppe/add_test_sense_method

comment:2 Changed 6 years ago by mkoeppe

  • Cc dimpase added
  • Commit set to a3a9a0f6cf826d27b2554e831133ddf2f303350d
  • Status changed from new to needs_review

New commits:

a3a9a0fGenericBackend._test_sense: New

comment:3 Changed 6 years ago by dimpase

We'll be away for 5 days, for a vacation...

comment:4 Changed 6 years ago by mkoeppe

Enjoy your vacation! This is not urgent.

comment:5 Changed 6 years ago by mkoeppe

Still needs review...

comment:6 Changed 6 years ago by dimpase

  • Milestone changed from sage-7.4 to sage-7.5
  • Reviewers set to Dima Pasechnik
  • Status changed from needs_review to positive_review

A lot of tests fail if I test with Nonexistent_LP_solver in --optional. Should all these tests be tagged differently? (another option would be to add a backend so named :-))

comment:7 Changed 6 years ago by vbraun

  • Branch changed from u/mkoeppe/add_test_sense_method to a3a9a0f6cf826d27b2554e831133ddf2f303350d
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.