Opened 2 years ago

Closed 2 years ago

#28766 closed enhancement (fixed)

Implement incidence matrix for cones

Reported by: gh-kliem Owned by:
Priority: major Milestone: sage-9.0
Component: geometry Keywords: cones, incidence matrix
Cc: jipilab, gh-LaisRast Merged in:
Authors: Jonathan Kliem Reviewers: Laith Rastanawi
Report Upstream: N/A Work issues:
Branch: 109b006 (Commits, GitHub, GitLab) Commit: 109b00684c60ea90d69dd23292f3eee87b0df28a
Dependencies: Stopgaps:

Status badges

Description

This ticket implements the incidence matrix for cones. This method is used to simplify getting the face lattice.

Columns correspond to facets, rows to rays. So there is no extra row for the vertex.

As a follow up, one can then obtain the combinatorial polyhedron of a cone.

Change History (5)

comment:1 Changed 2 years ago by gh-kliem

  • Branch set to public/28766
  • Commit set to 6c6c358737c456f643fd011936977e8d477f882a

New commits:

6c6c358implement incidence matrix for cones

comment:2 Changed 2 years ago by gh-kliem

  • Status changed from new to needs_review

comment:3 Changed 2 years ago by gh-kliem

  • Branch changed from public/28766 to public/28766-reb
  • Commit changed from 6c6c358737c456f643fd011936977e8d477f882a to 109b00684c60ea90d69dd23292f3eee87b0df28a

New commits:

e4abaa6implement incidence matrix for cones
109b006make incidence matrix immutable

comment:4 Changed 2 years ago by gh-LaisRast

  • Reviewers set to Laith Rastanawi
  • Status changed from needs_review to positive_review

I think this is good to go.

comment:5 Changed 2 years ago by vbraun

  • Branch changed from public/28766-reb to 109b00684c60ea90d69dd23292f3eee87b0df28a
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.