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

.show?? pops up the graphics item as well as the help page #2163

Closed
jasongrout opened this issue Feb 14, 2008 · 5 comments
Closed

.show?? pops up the graphics item as well as the help page #2163

jasongrout opened this issue Feb 14, 2008 · 5 comments

Comments

@jasongrout
Copy link
Member

At the command line:

sage: a=plot(x^2,(x,-1,1))
sage: a.show??
sage: a.show?

In either of the last two commands, the plot pops up on my Ubuntu 7.10 box (as well as the help).

Component: graphics

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

@mwhansen
Copy link
Contributor

comment:3

This is due to the following:

String Form:    <bound method Graphics.show of >

whereas after show_default(False), we get:

String Form:    <bound method Graphics.show of Graphics object consisting of 1 graphics primitive>

@mwhansen mwhansen assigned mwhansen and unassigned williamstein Feb 15, 2008
@mwhansen
Copy link
Contributor

comment:4

This can be fixed if we can tell IPython to set show_default(False) when doing things like repr(a.show)

@mwhansen mwhansen closed this as completed Sep 3, 2008
@mwhansen mwhansen reopened this Sep 3, 2008
@mwhansen
Copy link
Contributor

mwhansen commented Dec 2, 2008

comment:7

Attachment: trac_2163.patch.gz

I had already fixed some problems caused by removing the 'nodoctest' at the top before realizing there's no good way to test the function which I added.

The thing which addresses this ticket is the last hunk at the bottom of the patch.

@jasongrout
Copy link
Member Author

comment:8

This fixes the problem. Tests pass in interpreter.py as well (thanks for fixing all those too!)

Positive review.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Dec 5, 2008

comment:9

Merged in Sage 3.2.2.alpha0

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Dec 5, 2008
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