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
637 data model visualization is broken #638
Conversation
…and doesn't allow passing in a command anymore Signed-off-by: Nicolas Höning <nicolas@seita.nl>
…quired) Signed-off-by: Nicolas Höning <nicolas@seita.nl>
…replaces the --dev option Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Pull Request Test Coverage Report for Build 4722440452
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't know this feature and I really like it 😄
I run it locally and I an error when I had to install the required packages:
sudo apt install python3-pydot python3-pydot-ng graphviz
Error:
E: Unable to locate package python3-pydot-ng
By just installing graphviz
with apt
, the command make show-data-model
, run without problem. Maybe we should update line.
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
…hemadisplay library), rename a variable for readability Signed-off-by: Nicolas Höning <nicolas@seita.nl>
I believe you're right and I confimed it locally, @victorgarcia98. It seems the pydot capability can be also installed by pip, and actually it does come as a dependency of the library anyway. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! That would be it :D
Closes #637 check there for some investigation details.