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

Graphic.append() does not update xmin, xmax etc. #206

Closed
sagetrac-dmharvey mannequin opened this issue Jan 23, 2007 · 3 comments
Closed

Graphic.append() does not update xmin, xmax etc. #206

sagetrac-dmharvey mannequin opened this issue Jan 23, 2007 · 3 comments

Comments

@sagetrac-dmharvey
Copy link
Mannequin

sagetrac-dmharvey mannequin commented Jan 23, 2007

If I add a bunch of graphics primitives to a Graphic object using the Graphic append() method, the __xmax, __xmin, __ymax, __ymin attributes are not updated. Therefore when I try to plot the graphic, nothing shows up. Code for the correct behaviour can be found in the Graphic.__add__() method.

Component: packages: standard

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

@rlmill
Copy link
Mannequin

rlmill mannequin commented Aug 18, 2007

@rlmill rlmill mannequin closed this as completed Aug 18, 2007
@rlmill
Copy link
Mannequin

rlmill mannequin commented Aug 18, 2007

comment:3

The correct behaviour is not quite that in Graphic.add(), since this is adding two Graphics objects. The reason for this bug is that primitives don't usually remember their xmin/max values, and if they do it's inconsistent.

@rlmill rlmill mannequin reopened this Aug 18, 2007
@williamstein williamstein added this to the sage-2.8.2 milestone Aug 18, 2007
@williamstein
Copy link
Contributor

comment:5

After a massive amount of discussion, we decided that there should be no Graphic.append method. It just doesn't even make sense. So we removed it and closed this bug.

@williamstein williamstein assigned sagetrac-agc and unassigned rlmill Aug 19, 2007
tobiasdiez pushed a commit to tobiasdiez/sage that referenced this issue Feb 22, 2024
Add PyPI mappings for setuptools-scm and python-htcondor
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