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

Signal inter #45

Merged
merged 2 commits into from
Feb 22, 2019
Merged

Signal inter #45

merged 2 commits into from
Feb 22, 2019

Conversation

nilgoyette
Copy link
Collaborator

Updated docs, defaulted SI and removed SI management. It "fixes" #28, kind-of, but you might want to keep it open or to create a new and better explained task :) It makes write_nifti and write_rgb_nifti more alike, which is good if we want to merge them.

Btw, do you plan to do a release someday? We are interested in all the "writer" stuff, of course!

Copy link
Owner

@Enet4 Enet4 left a comment

Choose a reason for hiding this comment

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

This works.

I also agree that it's high time we had a release. Allow me to check the other PR and throw in a release this evening.

@Enet4 Enet4 merged commit c54f429 into Enet4:master Feb 22, 2019
@nilgoyette nilgoyette deleted the signal_inter branch June 4, 2020 13:26
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