Simplicial complexes: bug in add_face

sagemath / sage

Main repository of SageMath

https://www.sagemath.org

Other

1.38k stars 470 forks source link

Closed jhpalmieri closed 7 years ago

jhpalmieri commented 7 years ago

As reported at sage-devel, there is a bug with adding a face, once the attribute _faces has been populated:

sage: X = SimplicialComplex([[0], [1]])
sage: temp = X.faces(SimplicialComplex(()))
sage: X.add_face([0,1])

raises an error.

Component: algebraic topology

Author: John Palmieri

Branch/Commit: b6fe4b5

Reviewer: Christian Nassau

jhpalmieri commented 7 years ago

jhpalmieri commented 7 years ago

Commit: b6fe4b5

jhpalmieri commented 7 years ago

New commits:

b6fe4b5 trac 22880: fix bug with add_face for simplicial complexes

cnassau commented 7 years ago

Reviewer: Christian Nassau

cnassau commented 7 years ago

comment:3

This appears to be a pretty straightforward bugfix.

vbraun commented 7 years ago