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.

Sign up for GitHub

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

Jump to bottom

Fix ZeroDivisionError of Polyhedron.center() #27672

Closed
kliem opened this issue Apr 15, 2019 · 5 comments
Closed

Fix ZeroDivisionError of Polyhedron.center() #27672

kliem opened this issue Apr 15, 2019 · 5 comments
Labels
c: geometry r: wontfix t: bug

Comments

@kliem
Copy link
Contributor

kliem commented Apr 15, 2019 

sage: Polyhedron().center()
Traceback (most recent call last):
...
ZeroDivisionError: rational division by zero

I think None is a better output in this case.

CC: @embray @jdemeyer

Component: geometry

Keywords: polytopes, polyhedron, center

Issue created by migration from https://trac.sagemath.org/ticket/27672
@kliem kliem added this to the sage-8.8 milestone Apr 15, 2019
@kliem

This comment has been minimized.

Sign in to view
@kliem
Copy link
Contributor Author

kliem commented Apr 15, 2019

comment:2

This ticket can be closed, sorry.

I guess I should have really read the docstring.

It is pointed out there, that this will happen.

@kliem
Copy link
Contributor Author

kliem commented Apr 15, 2019

Changed author from Jonathan Kliem to none

@kliem

This comment has been minimized.

Sign in to view
@kliem kliem removed this from the sage-8.8 milestone Apr 15, 2019
@fchapoton
Copy link
Contributor

comment:4

please close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: geometry r: wontfix t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fchapoton @kliem