Skip to content

Removed/renamed DetTrackPAMFlow? #1794

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.

Sign up for GitHub

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

Jump to bottom
Closed
mgxd opened this issue Mar 25, 2019 · 1 comment · Fixed by nipy/nipype#2905
Closed

Removed/renamed DetTrackPAMFlow? #1794

mgxd opened this issue Mar 25, 2019 · 1 comment · Fixed by nipy/nipype#2905

Comments

@mgxd
Copy link

mgxd commented Mar 25, 2019

Description

This issue is popping up in nipype builds

https://github.com/nipy/nipype/blob/232ac23bfd48240906e7e629e98eec854bfdcb1c/nipype/interfaces/dipy/tracks.py#L23

Looks like version 0.16.0 removed dipy.workflows.tracking.DetTrackPAMFlow - has this been refactored to https://github.com/nipy/dipy/blob/193a8d95d9b5505a94c14b101c1aab49bb0ec43b/dipy/workflows/tracking.py#L21? cc @skoudoro
@skoudoro
Copy link
Member

Hi @mgxd,

Indeed, I need to make this smarter. I will create a PR this week to fix that.

thank you for pinging me

@mgxd mgxd mentioned this issue Mar 27, 2019
Merged
1 task
This was referenced Mar 28, 2019
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] Add all DIPY workflows dynamically skoudoro/nipype
2 participants
@mgxd @skoudoro