#13624: KeyError raised by GraphLatex.dot2tex_picture when edge_labels=True
------------------------------------------+---------------------------------
       Reporter:  slabbe                  |         Owner:  tbd       
           Type:  defect                  |        Status:  needs_work
       Priority:  major                   |     Milestone:  sage-5.9  
      Component:  packages: optional      |    Resolution:            
       Keywords:  dot2tex, graph drawing  |   Work issues:            
Report Upstream:  N/A                     |     Reviewers:            
        Authors:  Nicolas M. ThiƩry       |     Merged in:            
   Dependencies:                          |      Stopgaps:            
------------------------------------------+---------------------------------

Comment (by nthiery):

 Replying to [comment:27 ncohen]:
 > > See the method layout_graphviz in generic_graph.
 >
 > Do we both agree that this is nowhere sufficient ?
 >
 > Here is what happens when some code that requires graphviz (and which
 does not use layout_graphviz) is executed wihout graphviz installed
 >
 > {{{
 > sage: sage: import dot2tex
 # optional - dot2tex graphviz ## line 10 ##
 > sage: sage: output = dot2tex.dot2tex(graph, format="positions",
 prog="neato") # optional - dot2tex graphviz ## line 11 ##
 > ERROR    Could not locate Graphviz binaries
 > ERROR    Failed to create xdotdata. Is Graphviz installed?
 > ERROR    Failed to parse the input data. Is it a valid dot file?
 > Try to input xdot data directly. Example:
 >     dot -Txdot file.dot | dot2tex > file.tex
 >
 > If this does not work, check that you have an updated version of
 PyParsing and
 > Graphviz. Users have reported problems with old versions. You can also
 run
 > dot2tex in debug mode using the --debug option:
 >     dot2tex --debug file.dot
 > A file dot2tex.log will be written to the current directory with
 detailed
 > information useful for debugging.
 > An exception has occurred, use %tb to see the full traceback.
 >
 > SystemExit: 1
 >
 > To exit: use 'exit', 'quit', or Ctrl-D
 > }}}
 >
 > The error message should redirect the user toward a way to install
 graphviz I guess. And I have never met "`SystemExit`"

 I agree this is ugly, but that's a problem with dot2tex, not with Sage
 (feel free to report!). Any Sage code using dot2tex should call
 assert_have_doc2tex.

 > By the way, do we really need pyparsing ? I thought this did not
 > belong to Sage, and I needed it a couple of days ago.

 No clue.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13624#comment:29>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to