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

Enable coverage for Cython code #5184

Open
adrn opened this issue Jul 17, 2016 · 3 comments
Open

Enable coverage for Cython code #5184

adrn opened this issue Jul 17, 2016 · 3 comments

Comments

@adrn
Copy link
Member

adrn commented Jul 17, 2016

It seems like Cython supports Coverage analysis now?
http://docs.cython.org/src/tutorial/profiling_tutorial.html#enabling-coverage-analysis

I'm not very familiar with coverage, but to me it doesn't look like the Cython code in Astropy is included in the coverage report.

@adrn
Copy link
Member Author

adrn commented Jul 17, 2016

@nstarman
Copy link
Member

I believe codecov, the current system, does this already.

@saimn
Copy link
Contributor

saimn commented Oct 29, 2021

We have coverage for C code but not for Cython. I did it for astroscrappy (astropy/astroscrappy#49), I guess something similar could be used for astropy.

@nstarman nstarman removed the Close? label Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants