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

Library of polytopes should allow specifying backend=... #21937

Closed
mkoeppe opened this issue Nov 23, 2016 · 14 comments
Closed

Library of polytopes should allow specifying backend=... #21937

mkoeppe opened this issue Nov 23, 2016 · 14 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Nov 23, 2016

In #20885, we are working on a new Polyhedron backend class, Polyhedron_normaliz

Unfortunately all of the examples in sage.geometry.polyhedron.library (such as polytopes.cross_polytope) do not allow passing a backend parameter, so we can't use them for testing.

CC: @videlec @fchapoton @dimpase @vbraun @w-bruns @jplab @mo271

Component: geometry

Keywords: days93, polytope

Author: Jean-Philippe Labbé

Branch: 5532e21

Reviewer: Travis Scrimshaw

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

@mkoeppe mkoeppe added this to the sage-7.5 milestone Nov 23, 2016
@jplab
Copy link

jplab commented Feb 28, 2018

Branch: u/jipilab/21937

@jplab
Copy link

jplab commented Feb 28, 2018

comment:3

Here's a first version. I already added tests on the examples where normaliz can do tests right away.


New commits:

7f299c2First version, added backend argument

@jplab
Copy link

jplab commented Feb 28, 2018

Commit: 7f299c2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2018

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

5532e21pep8

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 28, 2018

Changed commit from 7f299c2 to 5532e21

@jplab
Copy link

jplab commented Feb 28, 2018

Author: Jean-Philippe Labbé

@jplab
Copy link

jplab commented Feb 28, 2018

New commits:

5532e21pep8

@jplab jplab modified the milestones: sage-7.5, sage-8.2 Feb 28, 2018
@tscrim
Copy link
Collaborator

tscrim commented Mar 1, 2018

comment:6

LGTM.

@tscrim
Copy link
Collaborator

tscrim commented Mar 1, 2018

Reviewer: Travis Scrimshaw

@jplab
Copy link

jplab commented Mar 2, 2018

Changed keywords from none to days93, polytope

@jplab
Copy link

jplab commented Mar 2, 2018

comment:8

Thanks Travis!

@vbraun
Copy link
Member

vbraun commented Mar 4, 2018

Changed branch from u/jipilab/21937 to 5532e21

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Sep 30, 2018

Changed commit from 5532e21 to none

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Sep 30, 2018

comment:10

Follow-up: #26365

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

4 participants