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

Integrate content from busiing:master #6

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

p-snft
Copy link
Member

@p-snft p-snft commented Oct 5, 2020

Replaces #3, which broke the original plotting functionality.

@pep8speaks
Copy link

Hello @p-snft! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 172:80: E501 line too long (118 > 79 characters)
Line 178:28: E231 missing whitespace after ','
Line 181:80: E501 line too long (115 > 79 characters)
Line 183:30: E231 missing whitespace after ','
Line 187:80: E501 line too long (115 > 79 characters)
Line 203:80: E501 line too long (81 > 79 characters)
Line 206:80: E501 line too long (80 > 79 characters)
Line 209:80: E501 line too long (81 > 79 characters)
Line 225:80: E501 line too long (100 > 79 characters)
Line 230:80: E501 line too long (118 > 79 characters)
Line 244:80: E501 line too long (85 > 79 characters)
Line 247:21: E231 missing whitespace after ','
Line 247:26: E231 missing whitespace after ','
Line 250:80: E501 line too long (144 > 79 characters)
Line 418:80: E501 line too long (86 > 79 characters)
Line 493:80: E501 line too long (89 > 79 characters)
Line 496:80: E501 line too long (102 > 79 characters)
Line 502:80: E501 line too long (89 > 79 characters)

@p-snft
Copy link
Member Author

p-snft commented Oct 6, 2020

@busiing: I tried to integrate this but I am a bit lost. Can you provide an example on how to use this? (As I understand it, it should somehow automatically create plots or many nodes within one figure - and also speed up the rendering process.)

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