Skip to content

Conversation

hveravallaskreppur
Copy link

please accept changes to spm_func.py with changes: added code for spm_spm_global sfunction :)

spm_funcs.py Outdated
img = nib.load(fname) # Load image
data = img.get_fdata() # Get image data

spm_values = np.asarray([]) # create empty array for spm values per volume
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can delete this line - you aren't using it.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

of course, thank you. I started a for loop then converted to a list comprehension and forgot to del. Fixing...

Copy link
Author

@hveravallaskreppur hveravallaskreppur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deleted one unnecssary line, as per your comment

spm_funcs.py Outdated
img = nib.load(fname) # Load image
data = img.get_fdata() # Get image data

spm_values = np.asarray([]) # create empty array for spm values per volume
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

of course, thank you. I started a for loop then converted to a list comprehension and forgot to del. Fixing...

@hveravallaskreppur
Copy link
Author

a question: I have now commited and pushed changes, is that then part of the same pull request or do I have to close this one and create a new one?

@matthew-brett
Copy link
Contributor

You make a pull request from a branch in your fork - in this case spm-funcs-fixes. Whenever you push new changes to that branch in your fork, the pull-request will update ...

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.

2 participants