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

Expose facet inequalities for lattice polytopes #8936

Closed
novoselt opened this issue May 9, 2010 · 5 comments
Closed

Expose facet inequalities for lattice polytopes #8936

novoselt opened this issue May 9, 2010 · 5 comments
Assignees
@sagetrac-mhampton
Labels
c: geometry p: major / 3 t: enhancement
Milestone
sage-4.4.4

Comments

@novoselt
Copy link
Member

novoselt commented May 9, 2010

While lattice polytopes compute and store facet normals and shifts internally, there were no functions to access them. This patch adds functions facet_normal and facet_constant to fix the situation.

I have also fixed a typo in the docstring of ReflexivePolytope, which has probably occurred during automatic conversion of docstrings.

CC: @vbraun

Component: geometry

Author: Andrey Novoseltsev

Reviewer: Volker Braun

Merged: sage-4.4.4.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/8936
@novoselt novoselt added this to the sage-4.4.4 milestone May 9, 2010
@novoselt
Copy link
Member Author

novoselt commented May 9, 2010

Author: Andrey Novoseltsev

@novoselt
Copy link
Member Author

novoselt commented May 9, 2010

Attachment: trac_8936_expose_facet_inequalities_for_lattice_polytopes.patch.gz

Fixed a mistake in the first version...

@vbraun
Copy link
Member

vbraun commented May 19, 2010

comment:2

Straight-forward addition of new functionality to the LatticePolytope class. Please merge...

@novoselt
Copy link
Member Author

novoselt commented Jun 7, 2010

Reviewer: Volker Braun

@mwhansen
Copy link
Contributor

mwhansen commented Jun 7, 2010

Merged: sage-4.4.4.alpha0

@mwhansen mwhansen closed this as completed Jun 7, 2010
This was referenced Jul 24, 2010
Closed
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-mhampton sagetrac-mhampton

Labels
c: geometry p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-4.4.4
Development

No branches or pull requests
3 participants
@mwhansen @vbraun @novoselt