Merge pull request #2110 from gadhvirushiraj/rushiDev

Fix version_table to work without Cython installed

doc/changes/2110.misc

@@ -0,0 +1 @@
+qutip.ipynbtools.version_table() can now be called without Cython installed

qutip/ipynbtools.py

@@ -39,10 +39,15 @@
 import qutip
 import numpy
 import scipy
-import Cython
 import matplotlib
 import IPython
 
+try:
+    import Cython
+    _cython_available = True
+except ImportError:
+    _cython_available = False
+
 
 def version_table(verbose=False):
     """
@@ -67,13 +72,14 @@ def version_table(verbose=False):
                 ("Numpy", numpy.__version__),
                 ("SciPy", scipy.__version__),
                 ("matplotlib", matplotlib.__version__),
-                ("Cython", Cython.__version__),
                 ("Number of CPUs", available_cpu_count()),
                 ("BLAS Info", _blas_info()),
                 ("IPython", IPython.__version__),
                 ("Python", sys.version),
                 ("OS", "%s [%s]" % (os.name, sys.platform))
                 ]
+    if _cython_available:
+        packages.append(("Cython", Cython.__version__))
 
     for name, version in packages:
         html += "<tr><td>%s</td><td>%s</td></tr>" % (name, version)

qutip/tests/test_ipynbtools.py

@@ -0,0 +1,24 @@
+from qutip.ipynbtools import version_table
+import pytest
+
+
+@pytest.mark.parametrize('verbose', [False, True])
+def test_version_table(verbose):
+    html_data = version_table(verbose=verbose).data
+    assert "<th>Software</th>" in html_data
+    assert "<th>Version</th>" in html_data
+    assert "<td>QuTiP</td>" in html_data
+    assert "<td>Numpy</td>" in html_data
+    assert "<td>SciPy</td>" in html_data
+    assert "<td>matplotlib</td>" in html_data
+    assert "<td>IPython</td>" in html_data
+    if verbose:
+        assert "<td>Installation path</td>" in html_data
+        if pytest.importorskip("getpass") is not None:
+            assert "<td>User</td>" in html_data
+
+
+@pytest.mark.skipif(not pytest.importorskip("Cython"), reason="cython not installed")
+def test_version_table_with_cython():
+    html_data = version_table().data
+    assert "<td>Cython</td>" in html_data