Ticket #3268 (closed defect: fixed)

Opened 5 years ago

Last modified 4 years ago

[with patch, positive review] Fix GAP interface printing

Reported by: rlm Owned by: mhansen
Priority: major Milestone: sage-3.3
Component: interfaces Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

I might be wrong, but it looks like output isn't getting printed: 

# From a pure GAP session:
GAP4, Version: 4.4.10 of 02-Oct-2007, i686-apple-darwin9.2.2-gcc
gap> g := Group((1,3,2),(2,4,3));
Group([ (1,3,2), (2,4,3) ])
gap> Stabilizer(g,4);
Group([ (1,3,2) ])
gap> 

# From a Sage session:
sage: %gap

  --> Switching to Gap <-- 

''
gap: g := Group((1,3,2),(2,4,3));

gap: Stabilizer(g,4);

gap:

Attachments

trac_3268.patch Download (824 bytes) - added by mhansen 4 years ago.

Change History

comment:1 Changed 5 years ago by mabshoff

  • Summary changed from Fix GAP interface to Fix GAP interface for Stabilizer()

comment:2 Changed 5 years ago by rlm

  • Summary changed from Fix GAP interface for Stabilizer() to Fix GAP interface printing

This isn't just for Stabilizer(), since the group isn't printing either...

Changed 4 years ago by mhansen

comment:3 Changed 4 years ago by mhansen

  • Owner changed from was to mhansen
  • Status changed from new to assigned
  • Summary changed from Fix GAP interface printing to [with patch, needs review] Fix GAP interface printing

Note that there is no good way to test this as you can't access the processed line as it's completely internal to the function.

comment:4 Changed 4 years ago by robertwb

  • Summary changed from [with patch, needs review] Fix GAP interface printing to [with patch, positive review] Fix GAP interface printing

Nice catch.

comment:5 Changed 4 years ago by mabshoff

  • Status changed from assigned to closed
  • Resolution set to fixed
  • Milestone changed from sage-3.4.1 to sage-3.3

Merged in Sage 3.3.alpha2.

Cheers,

Michael

Note: See TracTickets for help on using tickets.