Opened 3 years ago
Closed 3 years ago
#24849 closed enhancement (fixed)
Implement (sub)direct sums of polytopes
Reported by:  jipilab  Owned by:  

Priority:  major  Milestone:  sage8.3 
Component:  geometry  Keywords:  days93, polytope 
Cc:  vdelecroix, moritz, mkoeppe, chapoton  Merged in:  
Authors:  JeanPhilippe Labbé  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  ffe21e2 (Commits, GitHub, GitLab)  Commit:  ffe21e24d56b3508d32e3c63e35bd66a7b042b89 
Dependencies:  #24848, #22572  Stopgaps: 
Description (last modified by )
From https://www.csun.edu/~ctoth/Handbook/chap15.pdf:
The (d+d)dimensional polytope P⊕Q:=conv({(p,0)∈R^{(d+d')}p∈P}∪{(0,p')∈R^{(d+d')}q∈Q}). Thus the subdirect sum P⊕Q is a projection of the join P∗Q. The join is implemented in #24848.
Change History (12)
comment:1 Changed 3 years ago by
 Branch set to u/jipilab/24849
 Commit set to 6eb082501f7fee5e1fe659411ba4028c8a978beb
 Dependencies set to #24848, #22572
 Description modified (diff)
comment:2 Changed 3 years ago by
 Commit changed from 6eb082501f7fee5e1fe659411ba4028c8a978beb to b06397557db3de35303736e79faef44d07076ee9
Branch pushed to git repo; I updated commit sha1. New commits:
b063975  typo

comment:3 Changed 3 years ago by
 Status changed from new to needs_review
comment:4 Changed 3 years ago by
 Commit changed from b06397557db3de35303736e79faef44d07076ee9 to d0242f35e449ee63320eb52bddc285d47363625c
Branch pushed to git repo; I updated commit sha1. New commits:
d0242f3  removed translation from direct sum

comment:5 Changed 3 years ago by
 Commit changed from d0242f35e449ee63320eb52bddc285d47363625c to 0295def9ab8b166e553895c8e34d1d13a2104641
Branch pushed to git repo; I updated commit sha1. New commits:
0295def  whitespaces

comment:6 Changed 3 years ago by
 Commit changed from 0295def9ab8b166e553895c8e34d1d13a2104641 to 2457e9791c41ba408fee5295053cfb64f814b2a2
Branch pushed to git repo; I updated commit sha1. New commits:
2457e97  Conventions in docstring

comment:7 Changed 3 years ago by
 Commit changed from 2457e9791c41ba408fee5295053cfb64f814b2a2 to a1aabfa56ffc0694033e3c5c8653b6b87ba8f889
comment:8 Changed 3 years ago by
if the Author field is empty, no patchbot will look at the ticket..
comment:10 Changed 3 years ago by
 Commit changed from a1aabfa56ffc0694033e3c5c8653b6b87ba8f889 to ffe21e24d56b3508d32e3c63e35bd66a7b042b89
Branch pushed to git repo; I updated commit sha1. New commits:
ffe21e2  Merge branch 'sage8.2.b7' into 24849

comment:11 Changed 3 years ago by
 Milestone changed from sage8.2 to sage8.3
 Reviewers set to Frédéric Chapoton
 Status changed from needs_review to positive_review
ok, let it be
comment:12 Changed 3 years ago by
 Branch changed from u/jipilab/24849 to ffe21e24d56b3508d32e3c63e35bd66a7b042b89
 Resolution set to fixed
 Status changed from positive_review to closed
New commits:
first version of (sub)direct sum