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

Implementation of anatomical network visualisations #140

Open
wants to merge 43 commits into
base: master
Choose a base branch
from

Conversation

wingedRuslan
Copy link
Collaborator

@wingedRuslan wingedRuslan commented Jul 17, 2019

  • I'm ready to merge
  • What's the context for this pull request?
    (this is a good place to reference any issues that this PR addresses)

Created plotting anatomical network functionality
Issues: #126, #122

  • What's new?
    plot_anatomical_network function (+ some visualisation_helpers functions) and the tutorial to illustrate how to use new tools

  • What should a reviewer feedback on?
    Is documentation clear?
    Are the produced (and saved into a file) plots publication-ready and look nice?

  • Does anything need to be updated after merge?
    (e.g the wiki or the WhitakerLab website)
    Info on the website will be automatically updated

wingedRuslan and others added 30 commits April 2, 2019 23:12
…ing a TypeError if unsupported type provided
…rder to improve the process of saving a figure to file.
… change calc_nodal_measures in calculate_nodal_measures function
I've made a whole bunch of changes to the jupyter notebook...but they're all just "yes, and"-ing the work that wingedRuslan did!

So it looks like a big diff, and it kinda is, but very much only on a surface level <3
This is mostly a style and readability update...
I found the print outs of the warnings a little hard to parse in the jupyter notebook
and then I've changed the comments to better fit my understanding of what
was happening.
Makes sense to set those early,
especially if there's going to be a warning that folks want to understand.
Kirstie review of visualisation tutorial
@Islast
Copy link
Collaborator

Islast commented Jul 31, 2019

link to compare nx plotting branch to Ruslan master branch (eg PR 121)
All changes here seem absolutely fine

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.

3 participants