Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workaround for graphviz crash bug #47

Closed
wants to merge 1 commit into from
Closed

Conversation

kno10
Copy link
Collaborator

@kno10 kno10 commented Mar 28, 2018

This seemingly trivial change appears to avoid the graphviz crash bug.
But I don't know if this sometimes will produce less pretty plots.

@kno10
Copy link
Collaborator Author

kno10 commented Aug 14, 2018

Hi @dspinellis,
Would you have a look whether the port specification is needed at all (is the landingPort ever anything else than the entire class?), or whether we can add this workaround (using the center of the port, rather than the bounding box)?
But I noticed that my trimmed down test no longer crashes here. I will have to try the full version again.

@dspinellis
Copy link
Owner

The landing port can point to specific elements; see e.g. this example.

@kno10
Copy link
Collaborator Author

kno10 commented Oct 25, 2018

Does not appear to be necessary anymore for me. Closing.

@kno10 kno10 closed this Oct 25, 2018
@kno10 kno10 mentioned this pull request Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants