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

fix initializer for kamada_kawai_layout (networkx #3658) #3782

Merged
merged 5 commits into from
Jan 20, 2020

Conversation

boothby
Copy link
Contributor

@boothby boothby commented Jan 16, 2020

See discussion at #3658

@pep8speaks
Copy link

pep8speaks commented Jan 16, 2020

Hello @boothby! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-01-20 19:00:23 UTC

@boothby boothby force-pushed the master branch 2 times, most recently from 491ba56 to 4d94fa5 Compare January 16, 2020 23:31
@dschult
Copy link
Member

dschult commented Jan 19, 2020

This looks good. Anything else before we merge?

The smoke tests could all use removing vpos = for each test. That will improve the pycodestyle for the module too. If you don't get to it, I'll do that.
Thanks!

@boothby
Copy link
Contributor Author

boothby commented Jan 19, 2020

Absolutely; I just went along with the existing pattern, but I'll update the PR today or tomorrow.

@dschult dschult merged commit 15e17c0 into networkx:master Jan 20, 2020
kazimierz-256 pushed a commit to kazimierz-256/networkx that referenced this pull request Mar 20, 2020
…workx#3782)

* fixed high-dimensional initializer for kamada_kawai_layout (networkx networkx#3658)

* added tests for kamada_kawai 3d

* increased test coverage of kamada_kawai_layout

* fixed empty-graph oversight in kamada_kawai_layout

* removed unused variable vpos from smoke tests

Fixes networkx#3658
kiryph pushed a commit to kiryph/networkx that referenced this pull request Jul 21, 2020
…workx#3782)

* fixed high-dimensional initializer for kamada_kawai_layout (networkx networkx#3658)

* added tests for kamada_kawai 3d

* increased test coverage of kamada_kawai_layout

* fixed empty-graph oversight in kamada_kawai_layout

* removed unused variable vpos from smoke tests

Fixes networkx#3658
MridulS pushed a commit to MridulS/networkx that referenced this pull request Feb 4, 2023
…workx#3782)

* fixed high-dimensional initializer for kamada_kawai_layout (networkx networkx#3658)

* added tests for kamada_kawai 3d

* increased test coverage of kamada_kawai_layout

* fixed empty-graph oversight in kamada_kawai_layout

* removed unused variable vpos from smoke tests

Fixes networkx#3658
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants