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

Update to edge and nodestroke linewidths. #58

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

Conversation

rokkuran
Copy link

I noticed that that the edgelinewidth and nodestrokelw didn't play very nicely together. I added a LINEWIDTH parameter to replace EDGELINEWIDTH, which is used for both of the previous parameters.

edgelinewidth and nodestrokelw should function properly if used independently and together. When using arrays as inputs, each will be scaled relative to their maximum in each array.

I have attached a jupyter notebook with some examples of the change.

graphplot_linewidth.ipynb.tar.gz

note: this is actually my first time creating a pull request, so apologies if I've made some kind of breach of etiquette.

@rokkuran
Copy link
Author

rokkuran commented Apr 3, 2018

It looks like the checks failed when attempting to build the Homebrew package. I'm not sure why this has happened and haven't touched this before. When someone has time could they help clear this up for me?

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.

1 participant