cta-observatory / ctapipe

Low-level data processing pipeline software for CTAO or similar arrays of Imaging Atmospheric Cherenkov Telescopes

https://ctapipe.readthedocs.org

BSD 3-Clause "New" or "Revised" License

62 stars 265 forks source link

Impact code cleanup #2551

Closed gschwefer closed 2 months ago

gschwefer commented 2 months ago

In this PR I've collected all the implementations of the comments left on #2305 that I could just implement on the spot. This is mostly documentation updates, renaming of variables and such minor things. For the other comments, a bit more work or discussion might be required. I'm fine with either appending some of the things to this PR or opening new ones, depending on how large the things turn out to be

ctao-dpps-sonarqube[bot] commented 2 months ago

Passed

Analysis Details

2 Issues

0 Bugs
0 Vulnerabilities
2 Code Smells

Coverage and Duplications

81.30% Coverage (94.10% Estimated after merge)
0.00% Duplicated Code (0.60% Estimated after merge)

Project ID: cta-observatory_ctapipe_AY52EYhuvuGcMFidNyUs

View in SonarQube