Opened 10 years ago

Closed 7 years ago

#10126 closed enhancement (duplicate)

Fix error handing in Matrix_rational_dense._invert_pari()

Reported by: jdemeyer Owned by: jason, was
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: linear algebra Keywords: pari sig_on interrupt exception
Cc: Merged in:
Authors: Reviewers: Peter Bruin, Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by pbruin)

From sage/matrix/matrix_rational_dense.pyx:

# unfortunately I can't get signal handling to be good enough
# to properly catch error (and clean up) when trying to
# compute inverse, so we have to compute rank.  This does add
# time... (!) :-(

This is fixed by #15124.

Attachments (1)

trac_10126-invert_rational_matrix_pari.patch (2.5 KB) - added by pbruin 7 years ago.
fix inversion of rational matrices using PARI

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by pbruin

  • Cc pbruin added

Changed 7 years ago by pbruin

fix inversion of rational matrices using PARI

comment:2 Changed 7 years ago by pbruin

  • Authors set to Peter Bruin
  • Cc pbruin removed
  • Dependencies set to #14894
  • Description modified (diff)
  • Status changed from new to needs_review

comment:3 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:4 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.12 to sage-pending

comment:5 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-pending to sage-duplicate/invalid/wontfix

This is also fixed by #15124.

comment:6 Changed 7 years ago by pbruin

  • Authors changed from Peter Bruin to Jeroen Demeyer
  • Description modified (diff)
  • Reviewers set to Peter Bruin
  • Status changed from needs_review to positive_review

comment:7 Changed 7 years ago by jdemeyer

  • Authors Jeroen Demeyer deleted
  • Dependencies #14894 deleted
  • Resolution set to duplicate
  • Reviewers changed from Peter Bruin to Peter Bruin, Jeroen Demeyer
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.