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

Set up prism with both Vrep and Hrep #32150

Closed
kliem opened this issue Jul 7, 2021 · 6 comments
Closed

Set up prism with both Vrep and Hrep #32150

kliem opened this issue Jul 7, 2021 · 6 comments

Comments

@kliem
Copy link
Contributor

kliem commented Jul 7, 2021

We set up the prism over a polyhedron with the double description.

Before:

sage: P = polytopes.permutahedron(6)                                                                                                                                                
sage: %time Q = P.prism()                                                                                                                                                           
CPU times: user 231 ms, sys: 7.92 ms, total: 239 ms
Wall time: 237 ms
sage: P = polytopes.hypercube(8)                                                                                                                                                    
sage: %time Q = P.prism()                                                                                                                                                           
CPU times: user 65.3 ms, sys: 28 µs, total: 65.3 ms
Wall time: 63.9 ms
sage: P = polytopes.cross_polytope(8)                                                                                                                                               
sage: %time Q = P.prism()                                                                                                                                                           
CPU times: user 39.4 ms, sys: 0 ns, total: 39.4 ms
Wall time: 38.6 ms
sage: P = polytopes.hypercube(6, backend='field')                                                                                                                                   
sage: %time Q = P.prism()                                                                                                                                                           
CPU times: user 3.47 s, sys: 3.97 ms, total: 3.48 s
Wall time: 3.48 s

After:

sage: P = polytopes.permutahedron(6)                                                                                                                                                                                                                                           
sage: %time Q = P.prism()                                                                                                                                                                                                                                                      
CPU times: user 56.7 ms, sys: 3.85 ms, total: 60.6 ms
Wall time: 59.8 ms
sage: P = polytopes.hypercube(8)                                                                                                                                                                                                                                               
sage: %time Q = P.prism()                                                                                                                                                                                                                                                      
CPU times: user 26.5 ms, sys: 0 ns, total: 26.5 ms
Wall time: 26.2 ms
sage: P = polytopes.cross_polytope(8)                                                                                                                                                                                                                                          
sage: %time Q = P.prism()                                                                                                                                                                                                                                                      
CPU times: user 33.7 ms, sys: 0 ns, total: 33.7 ms
Wall time: 33 ms
sage: P = polytopes.hypercube(6, backend='field')                                                                                                                                                                                                                              
sage: %time Q = P.prism()                                                                                                                                                                                                                                                      
CPU times: user 14.1 ms, sys: 0 ns, total: 14.1 ms
Wall time: 13.8 ms
sage: P = polytopes.hypercube(10, backend='field')                                                                                                                                                                                                                             
sage: %time Q = P.prism()                                                                                                                                                                                                                                                      
CPU times: user 32.3 ms, sys: 0 ns, total: 32.3 ms
Wall time: 31.8 ms

Component: geometry

Keywords: polyhedron, double description, precomputed, prism

Author: Jonathan Kliem

Branch/Commit: f75726d

Reviewer: Travis Scrimshaw

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

@kliem kliem added this to the sage-9.4 milestone Jul 7, 2021
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 7, 2021

Changed commit from 41a6951 to f75726d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 7, 2021

Branch pushed to git repo; I updated commit sha1. New commits:

0cfc146tiny improvements
f75726donly compute prism once

@tscrim
Copy link
Collaborator

tscrim commented Jul 9, 2021

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jul 9, 2021

comment:3

LGTM.

@kliem
Copy link
Contributor Author

kliem commented Jul 9, 2021

comment:4

Thank you.

@vbraun
Copy link
Member

vbraun commented Jul 24, 2021

Changed branch from u/gh-kliem/double_description_prism to f75726d

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