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
Warn users if they don't have FURY installed #1902
Conversation
And provide a link for installation.
This better addresses issues such as #1718 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for making it clearer! I will merge it after this small correction
dipy/viz/__init__.py
Outdated
'fury', | ||
"You do not have FURY installed. Some visualization functions" | ||
"might not work for you. For installation instructions, please visit: " | ||
"https://fury.gl/") | ||
|
||
|
||
if have_fury: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update this variable have_fury
to has_fury
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated here and elsewhere to has_fury
to be consistent with has_mpl
below.
Codecov Report
@@ Coverage Diff @@
## master #1902 +/- ##
=========================================
Coverage ? 85.41%
=========================================
Files ? 119
Lines ? 14309
Branches ? 2249
=========================================
Hits ? 12222
Misses ? 1577
Partials ? 510
|
Thank you for this @arokem |
And provide a link to information about install