Opened 6 years ago
Closed 6 years ago
#18779 closed enhancement (fixed)
polytopes.gosset_3_21 and graphs.GossetGraph
Reported by:  ncohen  Owned by:  

Priority:  major  Milestone:  sage6.8 
Component:  geometry  Keywords:  
Cc:  vdelecroix, dimpase  Merged in:  
Authors:  Nathann Cohen  Reviewers:  Dima Pasechnik 
Report Upstream:  N/A  Work issues:  
Branch:  6527f4b (Commits, GitHub, GitLab)  Commit:  6527f4b41f580d470c734f30c109bc25049bfb29 
Dependencies:  Stopgaps: 
Description
https://en.wikipedia.org/wiki/Gosset_graph https://en.wikipedia.org/wiki/3_21_polytope
The graph is not generated from the polytope, as it takes ~16 seconds.
Nathann
Change History (15)
comment:1 Changed 6 years ago by
 Branch set to public/18779
 Status changed from new to needs_review
comment:2 Changed 6 years ago by
 Commit set to c3195be870d62e0815b0fb5ba1502a87a9bb26b4
comment:3 followup: ↓ 5 Changed 6 years ago by
 add gosset to the list of polytopes at the beginning of file
 use the correct fvector, even if not tested
comment:4 Changed 6 years ago by
 Commit changed from c3195be870d62e0815b0fb5ba1502a87a9bb26b4 to 2c3e33c797afda4b1cb6d222be542e3b91e0bc09
Branch pushed to git repo; I updated commit sha1. New commits:
2c3e33c  trac #18779: Reviewer's (dead right) comments

comment:5 in reply to: ↑ 3 Changed 6 years ago by
 add gosset to the list of polytopes at the beginning of file
 use the correct fvector, even if not tested
Both done. Sorry.
Nathann
comment:6 Changed 6 years ago by
 Commit changed from 2c3e33c797afda4b1cb6d222be542e3b91e0bc09 to 24ea250eeb70dfeacf881ae8622e7f42a908a38c
Branch pushed to git repo; I updated commit sha1. New commits:
bfdd09d  Merge branch 'public/gap478' of git://trac.sagemath.org/sage into develop

9a9458a  Merge branch 'public/18626' of git://trac.sagemath.org/sage into develop

4211dc8  Merge branch 'u/dimpase/18827' of git://trac.sagemath.org/sage into develop

4ab08fc  Merge branch 'u/dimpase/18830' of git://trac.sagemath.org/sage into develop

43f713c  Merge branch 'develop' of git://trac.sagemath.org/sage into develop

1023880  Merge remotetracking branch 'trac/public/18779' into gosset

24ea250  capitalise Gosset

comment:7 followup: ↓ 10 Changed 6 years ago by
oops, as usual, I only meant to push the last commit. After this change, I'd be happy to set it to positive review.
comment:8 Changed 6 years ago by
 Commit changed from 24ea250eeb70dfeacf881ae8622e7f42a908a38c to 1b391ddfcd53789652805eb8fcb2929338567c04
comment:9 Changed 6 years ago by
 Commit changed from 1b391ddfcd53789652805eb8fcb2929338567c04 to 6527f4b41f580d470c734f30c109bc25049bfb29
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
6527f4b  capitalise Gosset

comment:10 in reply to: ↑ 7 Changed 6 years ago by
oops, as usual, I only meant to push the last commit.
Fixed.
After this change, I'd be happy to set it to positive review.
I also modified the graph doctest which calls the polytope. Good to go?
Nathann
comment:11 followup: ↓ 12 Changed 6 years ago by
 Status changed from needs_review to positive_review
Looks good.
These 16 sec to build the graph from polytope highlight the need to have a method to do so directly, without constructing all faces. In fact, it can be done by just calling an LP solver the number of times equal to the number of pairs of vertices...
comment:12 in reply to: ↑ 11 Changed 6 years ago by
These 16 sec to build the graph from polytope highlight the need to have a method to do so directly, without constructing all faces.
I did that in #18860, but... Well, the function does not work :P
Actually, it works to build this polyhedron, but I hardcoded a constant '26'. If you can figure out what should appear instead of 26, then we will proably have a huge speedup for this .graph function :P
Nathann
comment:13 Changed 6 years ago by
 Status changed from positive_review to needs_work
Reviewer name...
comment:14 Changed 6 years ago by
 Reviewers set to Dima Pasechnik
 Status changed from needs_work to positive_review
comment:15 Changed 6 years ago by
 Branch changed from public/18779 to 6527f4b41f580d470c734f30c109bc25049bfb29
 Resolution set to fixed
 Status changed from positive_review to closed
Branch pushed to git repo; I updated commit sha1. New commits:
trac #18779: polytopes.gosset_3_21 and graphs.GossetGraph