Ticket #4540 (closed defect: fixed)

Opened 5 years ago

Last modified 4 years ago

[with patch; positive review] Symmetrica segfault converting Schur functions to k-Schurs

Reported by: jbandlow Owned by: mhansen
Priority: blocker Milestone: sage-3.2.1
Component: combinatorics Keywords: kschur, symmetrica, segfault
Cc: sage-combinat Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Example: 

sage: ks3 = kSchurFunctions(QQ,3,1)  # k-Schur functions without a 't' variable
sage: s = SFASchur(base_ring())
sage: ks3(s([3]))

Exception exceptions.TypeError: 'cannot convert a (= 1) to OP' in 'sage.libs.symmetrica.symmetrica._op_schur_general_dict' ignored
function: mult(1) 
python: mult(1): Unknown error 3052408646

I don't know if the definition of ks3 above was ever intended to be supported. I just tried it because I wanted k-Schur functions without a 't' and it seemed the natural thing to do. Conversions the other way (i.e., s(ks3([3])) ) do seem to work. And, whether it's intended to be supported or not, segfaults are bad.

Attachments

4540.patch Download (1.2 KB) - added by jbandlow 5 years ago.

Change History

Changed 5 years ago by jbandlow

comment:1 Changed 5 years ago by jbandlow

  • Summary changed from Symmetrica segfault converting Schur functions to k-Schurs to [with patch; needs review] Symmetrica segfault converting Schur functions to k-Schurs

comment:2 Changed 5 years ago by mabshoff

  • Priority changed from major to blocker

Segfaults are bad, so make this a blocker.

Mike: Can you have a look?

Cheers,

Michael

comment:3 Changed 5 years ago by mhansen

  • Status changed from new to assigned
  • Summary changed from [with patch; needs review] Symmetrica segfault converting Schur functions to k-Schurs to [with patch; positive review] Symmetrica segfault converting Schur functions to k-Schurs

Good work Jason!

comment:4 Changed 5 years ago by mabshoff

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

Merged in Sage 3.2.1.alpha0

comment:5 Changed 4 years ago by nthiery

  • Cc sage-combinat added; jbandlow removed
Note: See TracTickets for help on using tickets.