-
Notifications
You must be signed in to change notification settings - Fork 532
[FIX] Import error for nipype.interfaces.dipy.base #3414
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3414 +/- ##
=======================================
Coverage 65.21% 65.21%
=======================================
Files 307 307
Lines 40474 40477 +3
Branches 5351 5352 +1
=======================================
+ Hits 26395 26398 +3
Misses 13004 13004
Partials 1075 1075
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Co-authored-by: Chris Markiewicz <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
Thank you for the feedback @effigies and good catch for |
- Bump to commit from nipy/nipype#3414
This PR should fix the import issue with the DIPY interface. DIPY project introduces a new deprecator decorator which was not well handled by
inspect.getfullargspec()
. So, I just replaceinspect.getfullargspec()
byinspect.signature()
.fix #3411
fix dipy/dipy#2402
Could you test this PR @koenhelwegen @HippocampusGirl? Thank you.
Acknowledgment