Opened 5 years ago

Closed 5 years ago

#16233 closed defect (fixed)

Exceptions ignored by LeanMatrix operations

Reported by: yomcat Owned by:
Priority: major Milestone: sage-6.4
Component: matroid theory Keywords: interrupt
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: b051209 (Commits) Commit: b0512093f9307a34902e8d36c2f6f313bf6e8a5e
Dependencies: Stopgaps:

Description

Currently, various matroid operations do not stop on KeyboardInterrupt()

^CException KeyboardInterrupt: KeyboardInterrupt() in 'sage.matroids.lean_matrix.QuaternaryMatrix.set' ignored
^CException KeyboardInterrupt: KeyboardInterrupt() in 'sage.matroids.lean_matrix.QuaternaryMatrix.add_multiple_of_row_c' ignored

Change History (9)

comment:1 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:2 Changed 5 years ago by jdemeyer

  • Keywords interrupt added; matroid removed
  • Summary changed from KeyboardInterrupt() ignored by matroid operations to KeyboardInterrupt ignored by matroid operations

comment:3 Changed 5 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/16233
  • Created changed from 04/25/14 03:50:32 to 04/25/14 03:50:32
  • Modified changed from 06/27/14 07:16:00 to 06/27/14 07:16:00

comment:4 Changed 5 years ago by jdemeyer

  • Authors set to Jeroen Demeyer
  • Commit set to 393e2a7cfecf8708cd3a8d12be3aebbe94389bf6
  • Priority changed from minor to major
  • Summary changed from KeyboardInterrupt ignored by matroid operations to Exceptions ignored by LeanMatrix operations

New commits:

393e2a7Add except clauses to LeanMatrix Cython code

comment:5 Changed 5 years ago by jdemeyer

  • Status changed from new to needs_review

comment:6 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:7 Changed 5 years ago by git

  • Commit changed from 393e2a7cfecf8708cd3a8d12be3aebbe94389bf6 to b0512093f9307a34902e8d36c2f6f313bf6e8a5e

Branch pushed to git repo; I updated commit sha1. New commits:

b051209Fix stupid typo

comment:8 Changed 5 years ago by vbraun

  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

comment:9 Changed 5 years ago by vbraun

  • Branch changed from u/jdemeyer/ticket/16233 to b0512093f9307a34902e8d36c2f6f313bf6e8a5e
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.