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

WIP: try out Stefan Behnel's cython coverage #578

Closed
wants to merge 2 commits into from

Conversation

@matthew-brett
Copy link
Member

commented Feb 20, 2015

Trying Stefan's coverage work for Cython; see:

http://blog.behnel.de/posts/coverage-analysis-for-cython-modules.html

@arokem

This comment has been minimized.

Copy link
Member

commented Feb 20, 2015

Happy days!

It looks like you'll need to update the wheelhouse for us to see the
results of this on travis, right?

On Fri, Feb 20, 2015 at 11:17 AM, Matthew Brett notifications@github.com
wrote:

Trying Stefan's coverage work for Cython; see:

http://blog.behnel.de/posts/coverage-analysis-for-cython-modules.html

You can view, comment on, or merge this pull request online at:

#578
Commit Summary

  • NF: try out Stefan Behnel's cython coverage

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#578.

@stefanv

This comment has been minimized.

Copy link
Contributor

commented Feb 20, 2015

I think you'll have to change the Travis script to install Coverage directly from the bitbucket repository at https://bitbucket.org/ned/coveragepy

@stefanv

This comment has been minimized.

Copy link
Contributor

commented Feb 20, 2015

Actually, that's not true; I see an alpha is out: https://pypi.python.org/pypi/coverage/4.0a5

@stefanv

This comment has been minimized.

Copy link
Contributor

commented Feb 20, 2015

RF: do cython linetracing for nogil code too
Add line tracing for nogil code.
@matthew-brett

This comment has been minimized.

Copy link
Member Author

commented Feb 26, 2015

The coverage needs absolute latest Cython master and Coverage master - I think the alpha release of coverage will not work. There are still some problems even so:

https://groups.google.com/forum/#!topic/cython-users/EgWSRVuNHlA

@matthew-brett matthew-brett changed the title NF: try out Stefan Behnel's cython coverage WIP: try out Stefan Behnel's cython coverage Feb 26, 2015

@Garyfallidis

This comment has been minimized.

Copy link
Member

commented Dec 30, 2015

Any progress here? Should we revisit later?

@Garyfallidis

This comment has been minimized.

Copy link
Member

commented Sep 8, 2016

No progress on this PR. Any ideas? Is it now easier to measure Cython coverage?

@stefanv

This comment has been minimized.

Copy link
Contributor

commented Sep 9, 2016

According to http://docs.cython.org/en/latest/src/tutorial/profiling_tutorial.html, coverage causes significant slowdown for Cython, so you'll have to add a separate matrix entry.

@arokem

This comment has been minimized.

Copy link
Member

commented Mar 15, 2019

Looks like this is not going anywhere. I'll close for now.

@arokem arokem closed this Mar 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.