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

Viz Tests failing #3085

Closed
skoudoro opened this issue Feb 27, 2024 · 5 comments · Fixed by #3087
Closed

Viz Tests failing #3085

skoudoro opened this issue Feb 27, 2024 · 5 comments · Fixed by #3087
Assignees
Labels
category:visualization everything related to viz

Comments

@skoudoro
Copy link
Member

Hi @maharshi-gor,

Your PR has been merged a bit too fast. Tests are not passing.

Can you address the error below:

dipy/viz/__init__.py:4: in <module>
    from dipy.viz.horizon.app import horizon
dipy/viz/horizon/app.py:14: in <module>
    from dipy.viz.horizon.util import (check_img_dtype, check_img_shapes,
E     File "/Users/skoudoro/devel/dipy/dipy/viz/horizon/util.py", line 89
E       return np.unique(data[*indices]).shape[0] <= 2
E                                     ^
E   SyntaxError: invalid syntax
=============================================== short test summary info ================================================
ERROR dipy/viz/horizon/tab/tests/test_base.py
ERROR dipy/viz/tests/test_apps.py
ERROR dipy/viz/tests/test_fury.py
ERROR dipy/viz/tests/test_regtools.py
ERROR dipy/viz/tests/test_streamline.py
ERROR dipy/viz/tests/test_util.py
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 6 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
================================================== 6 errors in 1.01s ===================================================

Thank you,

@skoudoro skoudoro added the category:visualization everything related to viz label Feb 27, 2024
@Garyfallidis
Copy link
Contributor

Great catch @skoudoro !!!

@maharshi-gor
Copy link
Member

maharshi-gor commented Feb 28, 2024

Hi @skoudoro I tried running those on my computer they run completely fine on my computer
Can you check for fury installation on the computer
============================= test session starts ==============================
platform darwin -- Python 3.11.5, pytest-7.4.2, pluggy-1.3.0
rootdir: /Users/maharshigor/Desktop/GRG/dipy
configfile: pyproject.toml
collected 21 items

dipy/viz/horizon/tab/tests/test_base.py .... [ 19%]
dipy/viz/tests/test_apps.py ..... [ 42%]
dipy/viz/tests/test_fury.py ..... [ 66%]
dipy/viz/tests/test_regtools.py . [ 71%]
dipy/viz/tests/test_streamline.py ... [ 85%]
dipy/viz/tests/test_util.py ... [100%]

============================= 21 passed in 42.22s ==============================
<<<PYTHON-EXEC-OUTPUT
Finished running tests!

@maharshi-gor maharshi-gor self-assigned this Feb 28, 2024
@skoudoro
Copy link
Member Author

Hi @maharshi-gor,

it is not related to FURY but python version. it fails with python 3.9 and pass with python 3.11.

we support python 3.9+. it needs to work with python3.9.

Can you look at this?

Thanks

@maharshi-gor
Copy link
Member

Hi @skoudoro you can check #3087 for resolved issue.

Thank you so much for the your feedback.

@maharshi-gor
Copy link
Member

I have also created environment with different python version to make sure that I test backward compatibility on my local till the time test cases gets fixed.

@skoudoro skoudoro linked a pull request Feb 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:visualization everything related to viz
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants