Skip to content

Camino out_file naming #1061

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

Merged
merged 4 commits into from
Feb 23, 2015
Merged

Camino out_file naming #1061

merged 4 commits into from
Feb 23, 2015

Conversation

mick-d
Copy link
Contributor

@mick-d mick-d commented Feb 21, 2015

In the track interface, no provision was made when the out_file name was specified to rename the out_file accordingly. This is now corrected.

@mick-d
Copy link
Contributor Author

mick-d commented Feb 23, 2015

Checking Travis output the build was actually fine. I believe everything is ok, what do you think @oesteban ?

@oesteban
Copy link
Contributor

@mick-d I re-run the failing test and now it's ok ;)

@mick-d
Copy link
Contributor Author

mick-d commented Feb 23, 2015

Great, thanks! Then it's ok to merge?

satra added a commit that referenced this pull request Feb 23, 2015
@satra satra merged commit 640660f into nipy:master Feb 23, 2015
@satra
Copy link
Member

satra commented Feb 23, 2015

missed a CHANGES file. i'll update it.

@mick-d
Copy link
Contributor Author

mick-d commented Feb 26, 2015

Thanks @satra. I did not want to clutter the CHANGES file with such a small change but glad to hear it is worth notifying.

@mick-d mick-d deleted the camino_dyadcorr branch July 22, 2015 19:16
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