Opened 8 years ago
Closed 8 years ago
#17171 closed enhancement (fixed)
Upgrade dot2tex to 2.9.0
Reported by:  tscrim  Owned by:  sagecombinat 

Priority:  major  Milestone:  sage6.8 
Component:  packages: optional  Keywords:  dot2tex 
Cc:  sagecombinat, nthiery, aschilling, bsalisbury1, jpflori  Merged in:  
Authors:  Travis Scrimshaw  Reviewers:  Frédéric Chapoton, Jeroen Demeyer 
Report Upstream:  N/A  Work issues:  
Branch:  f5a9e93 (Commits, GitHub, GitLab)  Commit:  f5a9e93c642095ed7757521f73c87b961d41582c 
Dependencies:  Stopgaps: 
Description (last modified by )
dot2tex 2.9.0 was release in May, 2014. Currently we are at 2.9.0dev, and we should make sure we're at the latest stable release.
Upstream tarball: https://pypi.python.org/packages/source/d/dot2tex/dot2tex2.9.0.tar.gz#md5=2dbaeac905424d0410751235bde4b8b2
Change History (35)
comment:1 Changed 8 years ago by
Authors:  → Travis Scrimshaw 

Branch:  → public/optional_spkg/dot2tex2.9.017171 
Commit:  → 005d70129c9478552fb5e4dde63ececbd117a967 
Description:  modified (diff) 
Status:  new → needs_review 
comment:2 Changed 8 years ago by
Cc:  jpflori added 

comment:3 Changed 8 years ago by
Milestone:  sage6.4 → sage6.6 

comment:5 Changed 8 years ago by
Reviewers:  → Frédéric Chapoton 

Status:  needs_review → needs_work 
with graphviz and this new dot2tex, some trivial doctest failures:
sage t optional=sage,dot2tex,graphviz ./graphs/dot2tex_utils.py ./graphs/graph_latex.py ./graphs/generic_graph.py ./categories/crystals.py ./combinat/perfect_matching.py ./combinat/crystals/crystals.py ./combinat/rigged_configurations/kleber_tree.py ./combinat/posets/linear_extensions.py ./combinat/posets/posets.py
sage t graphs/generic_graph.py # 6 doctests failed sage t combinat/posets/posets.py # 1 doctest failed
comment:6 Changed 8 years ago by
Milestone:  sage6.6 → sage6.8 

comment:7 Changed 8 years ago by
Commit:  005d70129c9478552fb5e4dde63ececbd117a967 → 08646109a599914ccb43ee90c94596402585f43e 

comment:8 Changed 8 years ago by
I fixed the trivially failing doctests (I also made the view()
example not tested because it opens a separate window). Looks like this will need to rebased on beta3, which I will do tonight.
comment:10 Changed 8 years ago by
Commit:  08646109a599914ccb43ee90c94596402585f43e → 1938b1eecc5e4ee4a65df51885a9a8efae7a1d71 

Branch pushed to git repo; I updated commit sha1. New commits:
1938b1e  Merge branch 'public/optional_spkg/dot2tex2.9.017171' into 6.8.b3

comment:11 Changed 8 years ago by
Commit:  1938b1eecc5e4ee4a65df51885a9a8efae7a1d71 → 8eef83dddd16841649015d7dfea1a32c62fd3e84 

Branch pushed to git repo; I updated commit sha1. New commits:
159fa6a  Merge branch 'develop' into public/optional_spkg/dot2tex2.9.017171

758cf08  Merge branch 'public/optional_spkg/dot2tex2.9.017171' of trac.sagemath.org:sage into public/optional_spkg/dot2tex2.9.017171

8eef83d  Removed blanklines.

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

I had done my merge, checking, and was about to push when you did your merge :P.
Turns out the <BLANKLINE>
's are not needed.
comment:13 Changed 8 years ago by
I have a failure:
sage t combinat/posets/posets.py ********************************************************************** File "combinat/posets/posets.py", line 1259, in sage.combinat.posets.posets.FinitePoset._latex_ Failed example: print P._latex_() #optional  dot2tex graphviz Expected: \begin{tikzpicture}[>=latex,line join=bevel,] %% \node (node_1) at (6.0...bp,57.0...bp) [draw,draw=none] {$2$}; \node (node_0) at (6.0...bp,7.0...bp) [draw,draw=none] {$1$}; \draw [black,>] (node_0) ..controls (6.0...bp,20.2...bp) and (6.0...bp,30.9...bp) .. (node_1); % \end{tikzpicture} Got: <BLANKLINE> \begin{tikzpicture}[>=latex,line join=bevel,] %% \node (node_1) at (6.0bp,57.0bp) [draw,draw=none] {$2$}; \node (node_0) at (6.0bp,7.0bp) [draw,draw=none] {$1$}; \draw [black,>] (node_0) ..controls (6.0bp,19.947bp) and (6.0bp,30.897bp) .. (node_1); % \end{tikzpicture} <BLANKLINE>
because of the 19 instead of 20
And moreover, it pops me out 10 pdf pictures in new windows..
I did
sage bt optional=sage,dot2tex,graphviz ./graphs/__init__.py ./graphs/dot2tex_utils.py ./graphs/graph.py ./graphs/generic_graph.py ./graphs/digraph.py ./graphs/graph_latex.py ./categories/crystals.py ./combinat/perfect_matching.py ./combinat/crystals/crystals.py ./combinat/partition.py ./combinat/posets/posets.py ./combinat/posets/linear_extensions.py ./combinat/rigged_configurations/kleber_tree.py
comment:14 Changed 8 years ago by
Commit:  8eef83dddd16841649015d7dfea1a32c62fd3e84 → b7632caa4fe14de27a22d61074bc22e88ff14f3a 

Branch pushed to git repo; I updated commit sha1. New commits:
b7632ca  Making the control points more forgiving.

comment:15 Changed 8 years ago by
That should make that doctest pass.
For better or worse, that was the current behavior before this ticket. However, do you want me to go through and make similar changes with opening external windows?
comment:16 Changed 8 years ago by
Small detail:
# not tested  dot2tex, opens external window
should probably be
# optional  dot2tex, not tested (opens external window)
or something.
comment:17 followup: 18 Changed 8 years ago by
well, maybe it would be good, if optionals tests are now going to be run more often..
comment:18 Changed 8 years ago by
Replying to chapoton:
well, maybe it would be good, if optionals tests are now going to be run more often..
graphviz
isn't optional (it's experimental), so graphviz
will not be tested by default.
comment:19 Changed 8 years ago by
Commit:  b7632caa4fe14de27a22d61074bc22e88ff14f3a → 7cee4abaf00aa344a6db7bdff0ff4f15e8d018f8 

Branch pushed to git repo; I updated commit sha1. New commits:
7cee4ab  Changing optional tests to not tested because they open an external window.

comment:20 Changed 8 years ago by
I've marked all applicable optional tests as per Jeroen's suggestion.
comment:21 Changed 8 years ago by
Status:  needs_review → needs_work 

Replace
optional  dot2tex, graphviz
by
optional  dot2tex graphviz
(like it was before)
comment:22 followup: 23 Changed 8 years ago by
I'm assuming you're refering to
# optional  dot2tex, graphviz, not tested (opens external window)
Do you want it like
# optional  dot2tex graphviz, not tested (opens external window)
(which looks somewhat strange) or
# optional  dot2tex graphviz not tested (opens external window)
(which looks even more strange) or what? I didn't add commas anywhere else that I could see...
comment:23 Changed 8 years ago by
Replying to tscrim:
Do you want it like
# optional  dot2tex graphviz, not tested (opens external window)
Yes
comment:24 Changed 8 years ago by
Commit:  7cee4abaf00aa344a6db7bdff0ff4f15e8d018f8 → 96cb142f440fb0423e829485bcfe526746430c33 

Branch pushed to git repo; I updated commit sha1. New commits:
96cb142  Removing commas.

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

ok, looks good, as far as I can tell.
comment:28 Changed 8 years ago by
Reviewers:  Frédéric Chapoton → Frédéric Chapoton, Jeroen Demeyer 

comment:29 Changed 8 years ago by
Status:  positive_review → needs_work 

src/sage/doctest/sources.py
fails
comment:30 Changed 8 years ago by
Commit:  96cb142f440fb0423e829485bcfe526746430c33 → 2eb080c0d39e7aa0f2840ffca5605abfc54e3c76 

Branch pushed to git repo; I updated commit sha1. New commits:
2eb080c  Fix doctest/sources failure.

comment:31 Changed 8 years ago by
Status:  needs_work → positive_review 

Fixed. Trivial failure (and there are similar results that are there).
comment:32 Changed 8 years ago by
Status:  positive_review → needs_work 

I missed this the last time, but there should be a comma (or other separator) at the @@@
below:
view(H, tight_page=True) # optional  dot2tex@@@ not tested (opens external window)
comment:33 Changed 8 years ago by
Commit:  2eb080c0d39e7aa0f2840ffca5605abfc54e3c76 → f5a9e93c642095ed7757521f73c87b961d41582c 

Branch pushed to git repo; I updated commit sha1. New commits:
f5a9e93  Adding a missing comma and reverting previous change.

comment:35 Changed 8 years ago by
Branch:  public/optional_spkg/dot2tex2.9.017171 → f5a9e93c642095ed7757521f73c87b961d41582c 

Resolution:  → fixed 
Status:  positive_review → closed 
All of our current patches no longer apply because either they were merged upstream or were changes to the testing framework (which has completely changed from the dev version). However I added a new patch which removes the
test_semicolon
because this is the known, but not yet fixed, Issue #5.New commits:
Upgrade dot2tex to 2.9.0.