You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Most RAPIDS projects measure coverage in Python. Though it appears not all measure code coverage in Cython. However there is lots of Cython code in RAPIDS. So it would be ideal to capture this in our code coverage statistics (as well as uncover parts of Cython code that is not currently covered in tests)
Here is an example from cuML where Cython code coverage was added: rapidsai/cuml#3111
The text was updated successfully, but these errors were encountered:
I'm asking because I don't know if it does, but if someone tests it and finds that it doesn't then yes certainly we should raise an issue. I just commented on the corresponding Cython/coverage/Meson issue since it looks like someone almost has that working. If someone has bandwidth to test this, that PR branch would probably be a good place to start if coverage doesn't work with the latest Cython out of the box.
Most RAPIDS projects measure coverage in Python. Though it appears not all measure code coverage in Cython. However there is lots of Cython code in RAPIDS. So it would be ideal to capture this in our code coverage statistics (as well as uncover parts of Cython code that is not currently covered in tests)
Here is an example from cuML where Cython code coverage was added: rapidsai/cuml#3111
The text was updated successfully, but these errors were encountered: