Opened 8 years ago

Closed 8 years ago

#15717 closed defect (fixed)

Add some sig_on() in Singular library

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-6.2
Component: interfaces Keywords:
Cc: ppurka Merged in:
Authors: Jeroen Demeyer Reviewers: Punarbasu Purkayastha
Report Upstream: N/A Work issues:
Branch: u/jdemeyer/ticket/15717 (Commits, GitHub, GitLab) Commit: 13573a935764c03692d67661fbc31fa3cedf1349
Dependencies: #14333 Stopgaps:

Status badges

Description

This hangs (see #15710) and cannot be interrupted:

sage: F.<a>=CyclotomicField(49); G = F['z,w'].fraction_field(); 1/G(a^28 + a^23 - 1)

The attached branch fixes the "cannot be interrupted" part of the bug.

Change History (8)

comment:1 Changed 8 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/15717
  • Created changed from 01/23/14 16:02:58 to 01/23/14 16:02:58
  • Modified changed from 01/23/14 16:02:58 to 01/23/14 16:02:58

comment:2 Changed 8 years ago by jdemeyer

  • Commit set to 13573a935764c03692d67661fbc31fa3cedf1349
  • Status changed from new to needs_review

New commits:

1753f06Update Singular to version 3-1-6.
536a31fUpdate Singular function interface to API changes in Singular 3.1.6.
dd46991Remove description of deleted patches.
0a702e9Merge commit 'dd469911d19ed7080fec' into ticket/15710
13573a9Add some sig_on() blocks to Singular interface

comment:3 Changed 8 years ago by jdemeyer

  • Dependencies set to #14333

comment:4 Changed 8 years ago by ppurka

Thanks, I just tested it and this works. But, we don't need #14333. I just cherry-picked the last commit on to develop and the interrupt worked very well. :)

comment:5 Changed 8 years ago by jdemeyer

Sure, but I started #15710 by checking whether #14333 would fix it and any fix for #15710 would probably need to be based on #14333. And I just copied that commit here. Given that #14333 has positive review, it's not a problem, right?

comment:6 Changed 8 years ago by ppurka

  • Reviewers set to Punarbasu Purkayastha
  • Status changed from needs_review to positive_review

Yeah. I guess it shouldn't matter.

comment:7 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:8 Changed 8 years ago by vbraun

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