Add __copy__ method to backend normaliz of Polyhedron
Currently, when copying a normaliz polyhedron, the _normaliz_cone is not copied appropriately:
sage: p = polytopes.hypercube(7,backend='normaliz')
sage: pcopy = copy(p)
sage: p == pcopy
True
sage: p is pcopy
False
sage: pcone = p._normaliz_cone
sage: pcopycone = pcopy._normaliz_cone
sage: pcone == pcopycone
True
sage: pcone is pcopycone
True
This ticket implements the __copy__ method in the normaliz backend by making use of the method NmzConeCopy of PyNormaliz. This way, a copy of a polyhedron with backend Normaliz contains a true copy of the normaliz cone.
Change History (10)
| Branch: |
→ public/28633
|
| Commit: |
→ ed7bfd2273c5c33cdd0be4eb92ada02f82e5790b
|
| Status: |
new →
needs_review
|
| Cc: |
Jean-Philippe Labbé added
|
| Description: |
modified (diff)
|
| Summary: |
Make a true copy of normaliz cone →
Add __copy__ method to backend normaliz of Polyhedron
|
| Description: |
modified (diff)
|
| Status: |
needs_review →
needs_work
|
| Commit: |
ed7bfd2273c5c33cdd0be4eb92ada02f82e5790b →
d30c08770deb29882a05fc0c49223696fed92cdc
|
| Status: |
needs_work →
needs_review
|
| Reviewers: |
→ Jean-Philippe Labbé, Travis Scrimshaw
|
| Status: |
needs_review →
positive_review
|
| Branch: |
public/28633 →
d30c08770deb29882a05fc0c49223696fed92cdc
|
| Resolution: |
→ fixed
|
| Status: |
positive_review →
closed
|
| Commit: |
d30c08770deb29882a05fc0c49223696fed92cdc
|
New commits:
polyhedra with backend normaliz make a copy of the cone when copying