Skip to content

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

@wingedRuslan wingedRuslan mentioned this pull request Aug 12, 2019
59 tasks
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