Skip to content
This repository has been archived by the owner on Nov 19, 2018. It is now read-only.

[63] Problem with empty labels #6

Open
prmtl opened this issue Mar 8, 2015 · 2 comments
Open

[63] Problem with empty labels #6

prmtl opened this issue Mar 8, 2015 · 2 comments

Comments

@prmtl
Copy link
Contributor

prmtl commented Mar 8, 2015

Reported by [email protected], 2012-01-10T10:19:15Z

What steps will reproduce the problem?
  1. Define node with an empty label:
    node = Node('PROCESS_END', shape='doublecircle', style='filled', label='', width='.15')

What is the expected output?
PROCESS_END [width=".15", style=filled, shape=doublecircle, label=""];
or better:
PROCESS_END [width=".15", style="filled", shape="doublecircle", label=""];

What do you see instead?

  1. I get:
    PROCESS_END [width=".15", style=filled, shape=doublecircle, label=];
  2. and the the exception while generating png output:
    Error: XXX.gv:17: syntax error near line 17
    context: PROCESS_END [width=".15", style=filled, shape=doublecircle, >>> label=] <<< ;
  3. and the result picture without the requested width of .15, and with a PROCESS_END label .
What version of the product are you using? On what operating system?

The case is with 1.0.28 release.

Please provide any additional information below.

Release 1.0.25 manages this case properly.

IMO all the attribute values should be quoted.
Originally assigned to [email protected]
From: https://code.google.com/p/pydot/issues/detail?id=63

@prmtl
Copy link
Contributor Author

prmtl commented Mar 8, 2015

Comment 1: [email protected], Jan 10, 2012
I see where that might be coming from. Although some attributes are handled better without quotes, I do see the problem with not having quotes for the empty label ;)
Will look into it.

@prmtl
Copy link
Contributor Author

prmtl commented Mar 8, 2015

Comment 2: [email protected], Jun 14, 2013
We have this problem also with Theano. Did a fix was found?

@prmtl prmtl changed the title [Issue 63] Problem with empty labels Problem with empty labels Mar 8, 2015
@prmtl prmtl changed the title Problem with empty labels [63] Problem with empty labels Mar 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant