Opened 2 years ago
Closed 2 years ago
#28743 closed enhancement (fixed)
Lattice Polytopes: Implement incidence matrix
Reported by:  ghkliem  Owned by:  

Priority:  major  Milestone:  sage9.0 
Component:  geometry  Keywords:  polytopes, lattice polytopes 
Cc:  jipilab, ghLaisRast, novoselt  Merged in:  
Authors:  Jonathan Kliem  Reviewers:  Frédéric Chapoton, Laith Rastanawi 
Report Upstream:  N/A  Work issues:  
Branch:  14519a7 (Commits, GitHub, GitLab)  Commit:  14519a7fa49f50504149256c320ad893420ebbf3 
Dependencies:  #28741  Stopgaps: 
Description
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.
Change History (10)
comment:1 Changed 2 years ago by
 Branch set to public/28743
 Commit set to a100542b32b7ed5279f112c7b16a794fa0998e69
 Status changed from new to needs_review
comment:2 Changed 2 years ago by
There is also vertex_facet_pairing_matrix
, perhaps it is nice to crossreference these methods in documentation.
comment:3 Changed 2 years ago by
 Cc novoselt added
comment:4 Changed 2 years ago by
 Branch changed from public/28743 to public/28743reb
 Commit changed from a100542b32b7ed5279f112c7b16a794fa0998e69 to 3f0d6fe57a49406469f8ceb3d566166fbb35d0df
New commits:
3f0d6fe  implemted incidence matrix for lattice polytope

comment:5 Changed 2 years ago by
 Branch changed from public/28743reb to public/28743reb2
 Commit changed from 3f0d6fe57a49406469f8ceb3d566166fbb35d0df to 2d6093c88480001dc56e392df5a19d768347cc0b
comment:6 followup: ↓ 8 Changed 2 years ago by
missing empty line after + EXAMPLES::
comment:7 Changed 2 years ago by
 Commit changed from 2d6093c88480001dc56e392df5a19d768347cc0b to 14519a7fa49f50504149256c320ad893420ebbf3
Branch pushed to git repo; I updated commit sha1. New commits:
14519a7  missing empty line

comment:8 in reply to: ↑ 6 Changed 2 years ago by
comment:9 Changed 2 years ago by
 Reviewers set to Frédéric Chapoton, Laith Rastanawi
 Status changed from needs_review to positive_review
It is good to go.
comment:10 Changed 2 years ago by
 Branch changed from public/28743reb2 to 14519a7fa49f50504149256c320ad893420ebbf3
 Resolution set to fixed
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
New commits:
implemented incidence matrix and applied it