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

Graphics() should work in 3d as a valid empty object #4604

Closed
sagetrac-mhampton mannequin opened this issue Nov 24, 2008 · 6 comments
Closed

Graphics() should work in 3d as a valid empty object #4604

sagetrac-mhampton mannequin opened this issue Nov 24, 2008 · 6 comments

Comments

@sagetrac-mhampton
Copy link
Mannequin

sagetrac-mhampton mannequin commented Nov 24, 2008

From sage-support (and this has bugged me too):

I'm not sure if this is a bug or just something I'm misunderstanding,
but for 2D graphics I can write code like this.

g = Graphics()
g += line( [ [-1,-1], [1,1] ] )
g.show()

But in 3D if I do either

g = Graphics()
g += sphere( (1,1,1), 2 )
g.show()

or

g = sage.plot.plot3d.base.Graphics3dGroup()
g += sphere( (1,1,1), 2 )
g.show()

I get the error:

ValueError: min() arg is an empty sequence

Is there something I'm missing on how to create a graphics object and
add 3D graphics to it like the way it's done in 2D?

Component: graphics

Keywords: 3d

Author: Bill Cauchois

Reviewer: Tim Dumol

Merged: sage-4.3.1.rc1

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

@sagetrac-mhampton sagetrac-mhampton mannequin added this to the sage-4.3.1 milestone Nov 24, 2008
@sagetrac-wcauchois
Copy link
Mannequin

sagetrac-wcauchois mannequin commented Jan 16, 2010

Attachment: trac_4604.patch.gz

based on sage 4.3.1.alpha1

@sagetrac-wcauchois
Copy link
Mannequin

sagetrac-wcauchois mannequin commented Jan 16, 2010

comment:1

Robert and I confirmed this bug has been fixed in Sage 4.3. The attached patch implements a doctest for Graphics that implements this.

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 17, 2010

comment:2

LGTM.

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

Reviewer: Tim Dumol

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

Merged: sage-4.3.1.rc1

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 19, 2010

Author: Bill Cauchois

@rlmill rlmill mannequin removed the s: positive review label Jan 19, 2010
@rlmill rlmill mannequin closed this as completed Jan 19, 2010
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

1 participant