Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix GAP interface printing #3268

Closed
rlmill mannequin opened this issue May 21, 2008 · 5 comments
Closed

Fix GAP interface printing #3268

rlmill mannequin opened this issue May 21, 2008 · 5 comments

Comments

@rlmill
Copy link
Mannequin

rlmill mannequin commented May 21, 2008

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: 

Component: interfaces

Issue created by migration from https://trac.sagemath.org/ticket/3268

@rlmill rlmill mannequin added this to the sage-3.4.1 milestone May 21, 2008
@rlmill rlmill mannequin added c: interfaces labels May 21, 2008
@rlmill rlmill mannequin assigned williamstein May 21, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin changed the title Fix GAP interface Fix GAP interface for Stabilizer() May 25, 2008
@rlmill
Copy link
Mannequin Author

rlmill mannequin commented May 25, 2008

comment:2

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

@rlmill rlmill mannequin changed the title Fix GAP interface for Stabilizer() Fix GAP interface printing May 25, 2008
@mwhansen
Copy link
Contributor

Attachment: trac_3268.patch.gz

@mwhansen
Copy link
Contributor

comment:3

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.

@robertwb
Copy link
Contributor

comment:4

Nice catch.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 24, 2009

comment:5

Merged in Sage 3.3.alpha2.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.4.1, sage-3.3 Jan 24, 2009
@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Jan 24, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants