Eshin Jolly

yea I don't love the idea of forcing a `reset_index` incase the user has a meaningful index they want to retain. Definitely think we should mention it in the docs...

That's essentially what we're doing now. Right now each `Brain_Data` init grabs the **current** value of the nltools package-wide `MNI_Template` dictionary. It's convenient for setting it for a current working...

@ejolly Need to write some docs for this

We should include a minimum pytables version in our requirements which seems to fix this `pytables>=3.5.1`

But that's assuming we have `numpy>=1.16`. This might require more testing with different combinations of numpy and pytables...

@josiequita Can you provide a little bit more information about how you're calling ISC from within `nltools`? Also are you able to reproduce this on a non-cluster computer? Here's a...

Hmm this seems to be a matplotlib issue for you locally. It works just fine for me and I just added a test in cac65fa to run on my PR...

https://www.jiscmail.ac.uk/cgi-bin/webadmin?A2=fsl;70c96e5e.1404

Hi @kylestanley1313 it looks like it's due to your using an older version of `nilearn`. They made a [fix a few versions ago](https://github.com/nilearn/nilearn/commit/b36df65af7cb9926f5e4abc9ba3787353c90c349#diff-4a6505247f1580244d1efa080c79682b966d941c8f152583ae4653ae6bfc8461) to handle the new `sklearn` API. Since...

Related to #365