jakelishman / qutip

65%

master: 66%

LAST BUILD ON BRANCH dev.major-openmp

branch: dev.major-openmp

CHANGE BRANCH

x

Reset

• dev.major-openmp
• core-use-dispatchers
• data-layer-tests
• dev.major
• dev.major-fix-test-coverage
• dev.major-scipy-1.5-constraint
• dispatch-to-fix-weights
• faster-husimi-q
• fix-codeclimate-duplication-parser
• fix-https
• fix-isherm_csr
• fix-mkl-complexity
• fix-python-version-travis
• fix-python-version-travis-dev.major
• fix-some-pep8
• fix-tlist-c_ops
• fix-zcsr_isherm
• github-readme-ci-badge
• import-qip-subset-in-core
• licence
• master
• master-fix-test-coverage
• move-tests-to-github-actions
• packaging
• remove-vendored-qcircuit
• speed-up-add_csr
• v5.0.0a0

Build # 206

Build Type

push

travis-ci-com

Committed by jakelishman

Commit Message

Use cythonize not Cython.Distutil.build_ext

Cythonize handles dependencies between .pxd and .pyx files correctly
when cythonising multiple extension modules, whereas the (old)
Cython.Distutil.build_ext does not.  For example, if changes are made to
the Accumulator functions in `qutip/core/data/csr.pxd`, cythonize will
recognise that everything that cimports csr will need to be updated,
whereas the build_ext overrides will only re-cythonise csr.pyx.

As of Cython 0.29 there is a `new_build_ext` in the Cython source which
internally calls cythonize, but it's shadowed in the actual release.

Run Details

12462 of 19232 relevant lines covered (64.8%)

0.65 hits per line