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

Remove tracking interfaces #1054

Closed
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@arokem
Member

arokem commented May 21, 2016

This should supersede #1038, and removes the tracking.interfaces module altogether. It's been in limbo since as far back as 2012: 4ccf0bb

@arokem arokem added this to the 0.12 milestone May 21, 2016

@Garyfallidis

This comment has been minimized.

Member

Garyfallidis commented May 21, 2016

Thx @arokem This was an orphan for many years. And there is no plan to support traits. It would be good to have a thumbs up from @MrBago who wrote this code. So we can move ahead with the removal.
@arokem please also remove the dipy_tracking_gui command https://github.com/nipy/dipy/blob/master/bin/dipy_tracking_gui

@arokem

This comment has been minimized.

Member

arokem commented May 23, 2016

Merged a commit from @MrBago removing a couple more outdated modules, and removed the tracking gui script.

@MrBago MrBago referenced this pull request May 23, 2016

Merged

Remove tracking interfaces #1056

@MrBago

This comment has been minimized.

Contributor

MrBago commented May 23, 2016

See #1056.

@MrBago MrBago closed this May 23, 2016

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