Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement (sub)direct sums of polytopes #24849

Closed
jplab opened this issue Feb 26, 2018 · 23 comments
Closed

Implement (sub)direct sums of polytopes #24849

jplab opened this issue Feb 26, 2018 · 23 comments

Comments

@jplab
Copy link

jplab commented Feb 26, 2018

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.

Depends on #24848
Depends on #22572

CC: @videlec @mo271 @mkoeppe @fchapoton

Component: geometry

Keywords: days93, polytope

Author: Jean-Philippe Labbé

Branch/Commit: ffe21e2

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/24849

@jplab jplab added this to the sage-8.2 milestone Feb 26, 2018
@jplab
Copy link
Author

jplab commented Feb 27, 2018

Branch: u/jipilab/24849

@jplab
Copy link
Author

jplab commented Feb 27, 2018

New commits:

6eb0825first version of (sub)direct sum

@jplab
Copy link
Author

jplab commented Feb 27, 2018

Dependencies: #24848, #22572

@jplab
Copy link
Author

jplab commented Feb 27, 2018

Commit: 6eb0825

@jplab

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 27, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

b063975typo

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 27, 2018

Changed commit from 6eb0825 to b063975

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Changed commit from b063975 to d0242f3

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

d0242f3removed translation from direct sum

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

0295defwhitespaces

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Changed commit from d0242f3 to 0295def

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Changed commit from 0295def to 2457e97

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

2457e97Conventions in docstring

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

c807893first version of the join
a14d2c7pep8
347f99fMade docstring corrections
a1aabfaMerge branch '24848' into 24849

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 2, 2018

Changed commit from 2457e97 to a1aabfa

@fchapoton
Copy link
Contributor

comment:8

if the Author field is empty, no patchbot will look at the ticket..

@jplab
Copy link
Author

jplab commented Mar 2, 2018

Author: Jean-Philippe Labbé

@jplab
Copy link
Author

jplab commented Mar 2, 2018

comment:9

My bad... too many tickets at the same time.

That's good to know!!

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 6, 2018

Changed commit from a1aabfa to ffe21e2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 6, 2018

Branch pushed to git repo; I updated commit sha1. New commits:

ffe21e2Merge branch 'sage8.2.b7' into 24849

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:11

ok, let it be

@fchapoton fchapoton removed this from the sage-8.2 milestone Jun 25, 2018
@fchapoton fchapoton added this to the sage-8.3 milestone Jun 25, 2018
@vbraun
Copy link
Member

vbraun commented Jun 29, 2018

Changed branch from u/jipilab/24849 to ffe21e2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants