scilus / scilpy

The Sherbrooke Connectivity Imaging Lab (SCIL) Python dMRI processing toolbox
Other
54 stars 59 forks source link

codevov does not cov multithread processes #931

Open arnaudbore opened 4 months ago

arnaudbore commented 4 months ago

Some documentationl:

https://pytest-cov.readthedocs.io/en/latest/subprocess-support.html#if-you-use-multiprocessing-pool

https://github.com/nedbat/coveragepy/issues/883#issuecomment-650562896

Try some stuff but still doesnt work, thinking about raising an issue even tho this one seems to be very close to what we have

https://github.com/pytest-dev/pytest-cov/issues/139

arnaudbore commented 4 months ago

Exemple: https://app.codecov.io/gh/scilus/scilpy/blob/master/scilpy%2Freconst%2Fdivide.py#L266 Even it is tested with this script test: scripts/tests/test_btensor_metrics.py

AlexVCaron commented 4 months ago

Multiprocessing is not covered by pytest-cov anymore. They say use the configuration provided by coverage, here it is, hidden in the changelogs :

https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst#400-2022-09-28

Still not working on my end though ...