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

added spm_global function #37

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

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