Title: Publish C-level coverage of the CPython test suite · Issue #94759 · python/cpython · GitHub
Open Graph Title: Publish C-level coverage of the CPython test suite · Issue #94759 · python/cpython
X Title: Publish C-level coverage of the CPython test suite · Issue #94759 · python/cpython
Description: Feature or enhancement Automatically publish C-level coverage of the CPython test suite. Pitch Recent issues, such as the problems with frame.setlineno, have revealed that the C-level coverage of the test suite isn't perfect. While sourc...
Open Graph Description: Feature or enhancement Automatically publish C-level coverage of the CPython test suite. Pitch Recent issues, such as the problems with frame.setlineno, have revealed that the C-level coverage of t...
X Description: Feature or enhancement Automatically publish C-level coverage of the CPython test suite. Pitch Recent issues, such as the problems with frame.setlineno, have revealed that the C-level coverage of t...
Opengraph URL: https://github.com/python/cpython/issues/94759
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Publish C-level coverage of the CPython test suite","articleBody":"**Feature or enhancement**\r\n\r\nAutomatically publish C-level coverage of the CPython test suite.\r\n\r\n**Pitch**\r\n\r\nRecent issues, such as [the problems with frame.setlineno](https://github.com/python/cpython/issues/94438), have revealed that the C-level coverage of the test suite isn't perfect. While source-level coverage isn't the only metric of a good test suite, paying closer attention to it should help us have more confidence when making changes to the CPython core.\r\n\r\n**Proposed design for MVP**\r\n\r\n- **Publish results nightly:** This cadence should be enough for most cases. While various tools like codecov.io can run on every pull request, coverage takes a long time to run, and IME, there is enough noise in the results that developers (especially new to the project) would spend a lot of time \"chasing\" coverage.\r\n- **Use clang/llvm-cov rather than gcc/gcov:** As @erlend-aasland [pointed out](https://github.com/faster-cpython/ideas/issues/426#issuecomment-1178091681) the LLVM toolchain coverage provides branch counts and coverage of subexpressions which seems pretty useful. There is a risk that this isn't the same tooling as most Linux developers would use, and not what `make coverage-report` does.\r\n- **Publish to Github Pages:** The easiest thing seems to be to publish to Github Pages in a new repository in the `python` org for this purpose. This keeps all of the credential management within Github.\r\n\r\n**Out of scope**\r\n\r\nCreating diffs between coverage runs seems like it would be useful, but I haven't found a good tool for that yet.\r\n\r\nPython-level coverage reports would likewise be useful, but should be a follow-on to this.\r\n\r\n**Previous discussion**\r\n\r\n@brettcannon had set up [coverage reports in the past](https://github.com/python/cpython/actions/workflows/coverage.yml), but they have been broken for a couple of releases. This aims to be simpler than that approach by using a pure Github stack (not using codecov.io) and would just run nightly, not as part of every pull request.\r\n\r\nMore recent discussion began here: https://github.com/faster-cpython/ideas/issues/426\n\n\u003c!-- gh-pr-number: gh-94760 --\u003e\n* PR: gh-94760\n\u003c!-- /gh-pr-number --\u003e\n","author":{"url":"https://github.com/mdboom","@type":"Person","name":"mdboom"},"datePublished":"2022-07-11T19:24:58.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/94759/cpython/issues/94759"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:245c5181-33b0-1206-4d3f-91ae47d535ec |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | CF08:6FE17:14AEE30:1D50BF3:69691ACF |
| html-safe-nonce | 15a6affe10d6f344c1dfc53f0c9c0d504b260796b8325d2624c5d383e0370247 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDRjA4OjZGRTE3OjE0QUVFMzA6MUQ1MEJGMzo2OTY5MUFDRiIsInZpc2l0b3JfaWQiOiI4MzkyOTc0OTk1NjY2MDQ5NzQzIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 813a88df817049919be205169dce74dae45f695fc08bfe6cc09fa78f656322da |
| hovercard-subject-tag | issue:1301124602 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/94759/issue_layout |
| twitter:image | https://opengraph.githubassets.com/38fb80ad1baf4146f3915c6e51a47588ebe3641cff2c78d5f53c6efec6c431f4/python/cpython/issues/94759 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/38fb80ad1baf4146f3915c6e51a47588ebe3641cff2c78d5f53c6efec6c431f4/python/cpython/issues/94759 |
| og:image:alt | Feature or enhancement Automatically publish C-level coverage of the CPython test suite. Pitch Recent issues, such as the problems with frame.setlineno, have revealed that the C-level coverage of t... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | mdboom |
| hostname | github.com |
| expected-hostname | github.com |
| None | 0e60568924309a021b51adabdce15c2a2f285b556f3130d1a2fa2a5bce11c55f |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | dd206f7ed6207863172be4a783826e86bd2375c3 |
| ui-target | canary-1 |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width