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.

Sign up for GitHub

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

Jump to bottom

Use sysmon coverage #2697

Merged

maxnoe merged 2 commits into main from cov-sysmon

Feb 18, 2025

Member

maxnoe commented Feb 12, 2025 •

edited

Loading

Triggered by a discussion with ACADA people that they observe very bad performance under 3.12 and coverage testing, I found the option to use an "experimental" (what ever that actually means) new way of tracing coverage information available under 3.12 and above.

The differences seem to be quite stark:

Version	Pipeline Before	Pipeline Now	Tests Before	Tests Now
3.10	13:55	13:10	11:49	11:32
3.12	14:23	09:23	12:41	07:33
3.13	15:25	09:39	14:12	08:08

maxnoe added 2 commits

February 12, 2025 22:13


          try setting sysmon coverage

aa63abd


          precommit

4ffaca1

ctao-dpps-sonarqube bot commented Feb 12, 2025

Analysis Details

0 Issues

0 Bugs
0 Vulnerabilities
0 Code Smells

Coverage and Duplications

No coverage information (94.20% Estimated after merge)
No duplication information (0.70% Estimated after merge)

Project ID: cta-observatory_ctapipe_AY52EYhuvuGcMFidNyUs

View in SonarQube

maxnoe added the no-changelog-needed label

maxnoe changed the title ~~try setting sysmon coverage~~ Use sysmon coverage

maxnoe requested review from Tobychev, dneise and kosack

February 13, 2025 08:19

Tobychev approved these changes

View reviewed changes

kosack approved these changes

View reviewed changes

maxnoe merged commit 25314b3 into main

13 of 14 checks passed

maxnoe deleted the cov-sysmon branch

February 18, 2025 16:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog-needed