Opened 6 years ago

Closed 6 years ago

#3268 closed defect (fixed)

[with patch, positive review] Fix GAP interface printing

Reported by: rlm Owned by: mhansen
Priority: major Milestone: sage-3.3
Component: interfaces Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
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 (1)

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

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by mabshoff

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

comment:2 Changed 6 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 6 years ago by mhansen

comment:3 Changed 6 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 6 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 6 years ago by mabshoff

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

Merged in Sage 3.3.alpha2.

Cheers,

Michael

Note: See TracTickets for help on using tickets.