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

cdd backend fails to initialize empty polyhedron from double description #30330

Closed
kliem opened this issue Aug 10, 2020 · 6 comments
Closed

cdd backend fails to initialize empty polyhedron from double description #30330

kliem opened this issue Aug 10, 2020 · 6 comments
Labels
c: geometry p: critical / 2 t: bug
Milestone
sage-9.2

Comments

@kliem
Copy link
Contributor

kliem commented Aug 10, 2020 

sage: P1 = polytopes.regular_polygon(5, exact=False)
sage: P2 = Polyhedron()
sage: P1*P2
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
...
TypeError: can't multiply sequence by non-int of type 'NoneType'

We use the method _init_empty_polyhedron if applicable to fix this.

CC: @jplab @LaisRast

Component: geometry

Keywords: empty polyhedron, RDF, cdd

Author: Jonathan Kliem

Branch/Commit: c3896a2

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/30330
@kliem kliem added this to the sage-9.2 milestone Aug 10, 2020
@kliem
Copy link
Contributor Author

kliem commented Aug 10, 2020

New commits:

c3896a2fix initialization of impty RDF polyhedron from double descpription

@kliem
Copy link
Contributor Author

kliem commented Aug 10, 2020

Commit: c3896a2

@kliem
Copy link
Contributor Author

kliem commented Aug 10, 2020

Branch: public/30330

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 10, 2020

Reviewer: Matthias Koeppe

@kliem
Copy link
Contributor Author

kliem commented Aug 10, 2020

comment:3

Thank you.

@vbraun
Copy link
Member

vbraun commented Aug 12, 2020

Changed branch from public/30330 to c3896a2

@vbraun vbraun closed this as completed in a4b8705 Aug 12, 2020
This was referenced Jan 29, 2023
Open
Closed
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 p: critical / 2 t: bug
Projects
None yet
Milestone
sage-9.2
Development

No branches or pull requests
3 participants
@vbraun @mkoeppe @kliem