Skip to content

[ENH] Add new dipy interfaces #1090

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 31 commits into from
Feb 13, 2016
Merged

Conversation

oesteban
Copy link
Contributor

Added interfaces to RESTORE, response estimator and CSD in reconstructions. Added StreamlineTractography in tracks.py.

@oesteban
Copy link
Contributor Author

This is another PR to be reviewed by dipy's crew (@arokem, @Garyfallidis, etc)

@oesteban
Copy link
Contributor Author

@satra, @chrisfilo How could I check that dipy has certain version, and make the interfaces unavailable if too old?

@chrisgorgo
Copy link
Member

It would be great to add this, but there are still some tests failing.

@oesteban
Copy link
Contributor Author

hey @chrisfilo , yep, the problem is on checking the dipy version. My problem here is two-fold: 1) I don't know exactly what is the policy we want to set on dipy versions, since it is developing very (very) fast; and 2) I would need to check how you do this in other interfaces where version is important.

@oesteban oesteban changed the title Add new dipy interfaces [ENH] Add new dipy interfaces Sep 10, 2015
@oesteban
Copy link
Contributor Author

@satra @chrisfilo , tests are passing (circle hook didn't update, but passed), how do you see this?

chrisgorgo added a commit that referenced this pull request Feb 13, 2016
@chrisgorgo chrisgorgo merged commit 70a5128 into nipy:master Feb 13, 2016
@oesteban
Copy link
Contributor Author

Cool!

@oesteban oesteban deleted the enh/NewDipyInterfaces branch February 13, 2016 20:19
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 72.607% when pulling 6f61d0c on oesteban:enh/NewDipyInterfaces into c1e811e on nipy:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants