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

Lattice Polytopes: Implement incidence matrix #28743

Closed
kliem opened this issue Nov 15, 2019 · 17 comments
Closed

Lattice Polytopes: Implement incidence matrix #28743

kliem opened this issue Nov 15, 2019 · 17 comments
Labels
c: geometry p: major / 3 t: enhancement
Milestone
sage-9.0

Comments

@kliem
Copy link
Contributor

kliem commented Nov 15, 2019

This ticket implements the incidence matrix for lattice polytopes.

The columns correspond to facets, the rows to vertices.

It applies this method when computing the face lattice.

Depends on #28741

CC: @jplab @LaisRast @novoselt

Component: geometry

Keywords: polytopes, lattice polytopes

Author: Jonathan Kliem

Branch/Commit: 14519a7

Reviewer: Frédéric Chapoton, Laith Rastanawi

Issue created by migration from https://trac.sagemath.org/ticket/28743
@kliem kliem added this to the sage-9.0 milestone Nov 15, 2019
@kliem
Copy link
Contributor Author

kliem commented Nov 15, 2019

New commits:

a100542implemented incidence matrix and applied it

@kliem
Copy link
Contributor Author

kliem commented Nov 15, 2019

Branch: public/28743

@kliem
Copy link
Contributor Author

kliem commented Nov 15, 2019

Commit: a100542

@novoselt
Copy link
Member

comment:2

There is also vertex_facet_pairing_matrix, perhaps it is nice to cross-reference these methods in documentation.

@kliem
Copy link
Contributor Author

kliem commented Nov 19, 2019

Changed commit from a100542 to 3f0d6fe

@kliem
Copy link
Contributor Author

kliem commented Nov 19, 2019

New commits:

3f0d6feimplemted incidence matrix for lattice polytope

@kliem
Copy link
Contributor Author

kliem commented Nov 19, 2019

Changed branch from public/28743 to public/28743-reb

@kliem
Copy link
Contributor Author

kliem commented Dec 16, 2019

Changed branch from public/28743-reb to public/28743-reb2

@kliem
Copy link
Contributor Author

kliem commented Dec 16, 2019

Changed commit from 3f0d6fe to 2d6093c

@kliem
Copy link
Contributor Author

kliem commented Dec 16, 2019

New commits:

b81eaaeimplemted incidence matrix for lattice polytope
2d6093cmade incidence matrix cached an immutable

@fchapoton
Copy link
Contributor

comment:6

missing empty line after + EXAMPLES::

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 16, 2019

Changed commit from 2d6093c to 14519a7

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 16, 2019

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

14519a7missing empty line

@kliem
Copy link
Contributor Author

kliem commented Dec 16, 2019

comment:8

Sorry, missed that.

Replying to @fchapoton:

missing empty line after + EXAMPLES::

@LaisRast
Copy link

Reviewer: Frédéric Chapoton, Laith Rastanawi

@LaisRast
Copy link

comment:9

It is good to go.

@vbraun
Copy link
Member

vbraun commented Dec 20, 2019

Changed branch from public/28743-reb2 to 14519a7

@vbraun vbraun closed this as completed in 8e87b5d Dec 20, 2019
@kliem kliem mentioned this issue Apr 9, 2020
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: major / 3 t: enhancement
Projects
None yet
Milestone
sage-9.0
Development

No branches or pull requests
5 participants
@vbraun @novoselt @fchapoton @kliem @LaisRast