ctapipe Bi-Weekly Digest — Activity since 2024-12-15

• cta-observatory/ctapipe repo issues

  • Sonarqube action for merge requests does not run on merged result but on branch itself #2668 [maxnoe] bug

    Describe the bug Our sonarqube action is running on the checked-out branch for merge requests while the coverage...

• cta-observatory/ctapipe repo pull_requests

  • add PointingCalculator #2623 [ctoennis] enhancement calibration new functionality

    I wanted to post this pull request here for the PointingCalculator. I have not finished adding unit tests for the...

  • Fix sonar for pull requests #2671 [maxnoe] maintenance no-changelog-needed documentation-only

    Note: I put this as documentation only since this workflow is only active once it reaches the main branch, so...

    • Tobychev

    • kosack

  • Stats calc tool #2628 [TjarkMiener] calibration

    This PR adds a generic stats-calculation tool utilizing the PixelStatisticsCalculator. Related #2542

    • mexanick

    • kosack: A common error is to have too small a chunk size, but this now results in a very ugly error and...

  • Muon parameters update #2670 [jstvdk]

    I added new functionality to the muon analysis as described in the Issue #2665 , specifically new fields to the...

    • kosack

      • kosack: Suggested change pixel_r =...

      • kosack: I don't think you need to convert and take the value here. Numpy works fine with unit...

    • mexanick: Is this the same as l1201 of ctapipe.containers? If so, why the docstrings are different?

      • jstvdk: They are the same, but I didn't want to duplicate the exact same text. Or should I just...

      • mexanick: Perhaps this function simply should return MuonParametersContainer? Also, the number of input...

      • jstvdk: Perhaps this function simply should return MuonParametersContainer? All this module in fact is...

    • mexanick: "image" is missing?

    • mexanick: Consider merging this and the above functions, implementing along the way unification of the return type.

      • jstvdk: Sorry, let me repeat if I understood correctly. You are proposing to merge all the methods into...

      • mexanick: yes

created at 2024-12-29 06:27 by dinghy 1.3.3.