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

[RTM] move missing values to the beginning of the confounds file #523

Merged
merged 6 commits into from
May 17, 2017

Conversation

chrisgorgo
Copy link
Contributor

It's more intuitive in context of using those for nuissance regression.

@effigies
Copy link
Member

This is beyond my pandas experience. I'm willing to take your word that this does what you say, though.

@chrisgorgo
Copy link
Contributor Author

Let me add a changelog entry

@effigies
Copy link
Member

Resolved conflicts in changelog. I can't cancel the resulting build for you, but there's no need for it to run to merge this.

@effigies effigies merged commit b02990d into nipreps:master May 17, 2017
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.

None yet

2 participants