Opened 8 years ago
Closed 8 years ago
#17969 closed enhancement (fixed)
A tutorial on How to draw polytope in Tikz
Reported by:  JeanPhilippe Labbé  Owned by:  

Priority:  minor  Milestone:  sage6.7 
Component:  geometry  Keywords:  tikz, polytope, plot, days64 
Cc:  Moritz Firsching, Vincent Knight  Merged in:  
Authors:  JeanPhilippe Labbé  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  67d3c0e (Commits, GitHub, GitLab)  Commit:  67d3c0e1ab0e055bd65d0cc9c8a621eb4939e9b3 
Dependencies:  Stopgaps: 
Description
I prepared a tutorial to show how to use the method .tikz() of a polytope.
You can visualize a version at http://www.math.huji.ac.il/~labbe/polytope.html
This tutorial is meant to make the previous page more permanent.
Change History (18)
comment:1 Changed 8 years ago by
Branch:  → u/jipilab/tutotikz 

comment:2 Changed 8 years ago by
Commit:  → aef237afa83bc8332e616e1bbbcef157ae582f35 

comment:3 Changed 8 years ago by
Commit:  aef237afa83bc8332e616e1bbbcef157ae582f35 → 4ba288e383a612b1740c8c6899be9619357c9177 

Branch pushed to git repo; I updated commit sha1. New commits:
4ba288e  Corrected a typo

comment:4 Changed 8 years ago by
Status:  new → needs_review 

comment:5 Changed 8 years ago by
Helloooooo !
I just wanted to bring to your attention that we can now include pictures easily in Sage's doc. Just in case you would find it useful for this tutorial. The methods is explained there:
https://groups.google.com/d/topic/sagedevel/qR2t4LunLJg/discussion
Nathann
comment:8 Changed 8 years ago by
Authors:  → JeanPhilippe Labbé 

Keywords:  days64 added 
comment:9 Changed 8 years ago by
Status:  needs_work → needs_review 

comment:10 Changed 8 years ago by
Status:  needs_review → needs_work 

I would try to use the comment of Nathan to put images in the tutorial, I'm working on it currently.
comment:11 Changed 8 years ago by
Sorry, I thought you forgot to change it back. Sorry for the inconvenience.
comment:12 Changed 8 years ago by
Cc:  Vincent Knight added 

comment:13 Changed 8 years ago by
Commit:  4ba288e383a612b1740c8c6899be9619357c9177 → ef8f4986f3c62481eec62940cf40258d282383cc 

comment:14 Changed 8 years ago by
Status:  needs_work → needs_review 

Unfortunately, I believe adding tikz code to the documentation deserves more work than fulfilling the actual ticket. Therefore, I simply reverted to the previous version.
You may want to have a look at my try within the version "2d3f871". This was producing a picture when building the doc(!) and then putting the .py code in the .html. This is not really what is desired, but was a try.
I believe this thread in sage devel is perhaps good about this:
https://groups.google.com/forum/#!topic/sagedevel/qR2t4LunLJg
For now, including tikz code in the doc should deserve its own ticket I guess...
comment:15 Changed 8 years ago by
Branch:  u/jipilab/tutotikz → public/ticket/17969 

Commit:  ef8f4986f3c62481eec62940cf40258d282383cc → 67d3c0e1ab0e055bd65d0cc9c8a621eb4939e9b3 
Reviewers:  → Frédéric Chapoton 
Looks good to me. And very useful ! I have already used that at least twice using intructions on your web pages, and it is good to make it available as a tutorial.
If you agree with my review commit (small changes mainly), then please set to positive review.
New commits:
93b4ca0  Merge branch 'u/jipilab/tutotikz' into 6.6.rc2

67d3c0e  trac #17969 review commit

comment:16 Changed 8 years ago by
Status:  needs_review → positive_review 

Hi Frédéric!
Thanks a lot for the detailed review!
I agree with the changes and set to positive review.
comment:17 Changed 8 years ago by
Milestone:  sage6.6 → sage6.7 

comment:18 Changed 8 years ago by
Branch:  public/ticket/17969 → 67d3c0e1ab0e055bd65d0cc9c8a621eb4939e9b3 

Resolution:  → fixed 
Status:  positive_review → closed 
Branch pushed to git repo; I updated commit sha1. New commits:
Shortened the lines to fit convention