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

DOC - updated streamline_tools example with the LocalTracking Framework #1887

Merged
merged 1 commit into from Jul 19, 2019

Conversation

gabknight
Copy link
Contributor

Changed the call from EuDX to LocalTracking. EuDX is still the algorithm use here, but in the new framework.

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@4342302). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1887   +/-   ##
=========================================
  Coverage          ?   84.91%           
=========================================
  Files             ?      118           
  Lines             ?    14296           
  Branches          ?     2247           
=========================================
  Hits              ?    12139           
  Misses            ?     1645           
  Partials          ?      512

@skoudoro
Copy link
Member

skoudoro commented Jul 8, 2019

Thanks @gabknight. Should we keep the Eudx class or more generally eudx.py (old framework)?

@gabknight
Copy link
Contributor Author

@skoudoro, no, no need to keep it as it is essentially the same as LocalTracking with PeaksAndMetricDG. I want to refactor this: remove eudx.py, rename PeaksAndMetricDG to EudxDG, and move the eudx tests.

This will be in the another PR addressing #1501.

@skoudoro
Copy link
Member

skoudoro commented Jul 9, 2019

ok, sounds good! Do you need help? Today, I can do this part of removing eudx.py and renaming PeaksAndMetricsDG

@gabknight
Copy link
Contributor Author

@skoudoro sure give it a go if you have time. The tricky part will adjust the tests. It will be very good to do it, as the current PeakAndMetricDG is not properly tested.
If you don't, I will do it at the time RF the tracking.local package. I'll be away from now until 19.07, I'm not sure I can make it for the next release.

@skoudoro
Copy link
Member

I just run the documentation and this example, everything looks good. merging

Thanks @gabknight!

@skoudoro skoudoro merged commit df3b6ef into dipy:master Jul 19, 2019
@gabknight gabknight deleted the RF_streamline_example_eux branch April 4, 2024 20:27
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.

None yet

3 participants