ctapipe Bi-Weekly Digest — Activity since 2025-02-28

• cta-observatory/ctapipe repo issues

  • CameraFrame to TelescopeFrame #2704 [zhipzhang]

    I am quite curious about the conversion of cameraframe and TelescopeFrame. From the AltAz to telescopeframe...

    • zhipzhang: The implementation here should be correct! I misunderstood the implementation of this process.

  • Unable to create IRF from LST mono observations #2706 [rcervinoucm] bug

    Hello, I have trained and produced the DL2 files from an LST mono approach with CTLearn, and I and trying to produce...

    • maxnoe: You should still have a subarray prediction, even if the subarray only consists of a single telescope

    • TjarkMiener: You should still have a subarray prediction, even if the subarray only consists of a single...

  • Expose CORSIKA photons in the SimulatedEventContainer #2698 [vmharvey] enhancement

    Please describe the use case that requires this feature. I'm doing some studies that use the CORSIKA photon bunches...

    • vmharvey: Following on from cta-observatory/pyeventio#288, I think I can add on to this issue that support...

    • maxnoe: Would it be enough for you for now to just attach the simtel event to the ctapipe event? To...

  • Support writing high-precision timestamps to hdf5 #2428 [maxnoe] enhancement

    Please describe the use case that requires this feature. Currently, we write out all timestamps as TAI MJD. That is...

    • mdpunch: Three use cases on my side: With several telescopes, it should be possible to fit a plane (3...

  • Implement a standard analysis #977 [maxnoe] enhancement help wanted use case

    As discussed in Dortmund A Tool for DL0 → DL1 A Tool for splitting data into training / test sets A Tool for...

  • Make simulated shower distribution an interface for simulation event sources #1689 [maxnoe] good first issue module:IO

    Issue from a TODO in the dl1writer TODO: this needs to be fixed, since it currently requires access to...

    • maxnoe: Implemented in #2633

  • Remove defaultdict-like behavior from container map #2708 [maxnoe]

    This leads to confusing results from time to time as containers are happily created with the nan/default values in...

  • StereoMeanCombiner.predict_table() returns invalid table #2711 [TjarkMiener] bug

    Describe the bug The column "prefix_telescopes" is returned here with a list of tel ids which participated in the...

  • Do not use astropy sky coordinates for ground and tilted ground frame #1278 [maxnoe] bug design problem

    These are not sky coordinates in any sense, so they shouldn't be using astropy frames and SkyCoords

    • mdpunch: Found this (and the my linked issue) again, while googling to search for how to get the actual...

    • maxnoe: @mdpunch Sounds good. You could also think about directly contributing such algorithms in...

    • mdpunch: I suppose I could! Maybe as part of a coding sprint (to help if I get stuck deciding where to...

    • mdpunch: Found this (and the my linked issue) again, while googling to search for how to get the actual...

  • Absolute timestamps are not absolute, but relative to tilted plane (pointing direction) #2712 [mdpunch] bug

    Describe the bug The timestamps produced by SimTelEventSource are actually corrected for the pointing direction...

    • mdpunch: P.S. I can provide some code to undo those corrections.

    • mdpunch: Alternatively, similar code could be easily applied to the later data products (R0 onwards), to...

    • mdpunch: For the simtel files, then it could look like this: import eventio import numpy as np from...

• cta-observatory/ctapipe repo pull_requests

  • Hillas psi uncertainty ruo #2629 [ruoyushang]

    Adding estimation of image psi uncertainty and uncertainty of image cog along the rotated y-axis in hillas.py. Also...

    • ruoyushang: Hi @maxnoe @Tobychev , Is the processing time test sufficient for the approval? Best, Ruo

  • More ImPACT code cleanup #2705 [gschwefer]

    Hi everyone, I finally got around to clean up the ImPACT code a bit more. This PR has 3 major changes: The paths to...

  • Fix show config #2703 [maxnoe]

    • Tobychev

  • Store event times using CTAO high resolution timestamps #2707 [maxnoe]

    Fixes #2428

    • mdpunch: I checked on running this version on a couple of simtel files, and indeed now instead of having...

  • Muon parameters update #2670 [jstvdk] module:calib

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

    • mexanick: docs are failing due to an issue with the type hint resolution with a few similar...

    • maxnoe: Is the MuonRingContainer present in __all__ in containers.py?

    • maxnoe: ring_intensity_parameters?

      • mexanick: I was thinking about this change, but this function also computes the number of pixels in the...

      • maxnoe: Yes, I think the size here came from when everything related to intensity was called size. So...

    • maxnoe: This reads like you only modified the container, but you also actually implemented the...

    • maxnoe

    • maxnoe

    • kosack

  • Fix sonar for PRs from forks #2710 [maxnoe] no-changelog-needed

    Was failing as the PR information is not included in the github data, but we already included it before in the...

    • maxnoe: This is tested using the project here: cta-observatory/sonar-test#5 It will only take effect...

    • Tobychev

  • Fix checkout cleaning up coverage report in sonar ci #2713 [maxnoe] maintenance no-changelog-needed

    The checkout action by default removes all untracked files, which in this case removed the coverage report

    • maxnoe: Still not working, I am going back to testing in the test repo

    • Tobychev

    • maxnoe: Ok, now it's working

  • Add a method to read scheduling information with TableLoader #2709 [mdebony]

    In the class TableLoader, a method called read_scheduling_information was added to read scheduling block information...

    • maxnoe: Thanks, changes on its own look good, but: Please add a test For the other tables, we offer to...

    • maxnoe: I know it's inconsistent with the observation_info, but I I really don't like...

      • mdebony: Done

    • maxnoe: For better error messages, use np.testing.assert_array_equal(scheduling["sb_id"], [4, 1])

      • mdebony: Done

    • maxnoe: Thanks a lot

    • Tobychev

created at 2025-03-14 06:29 by dinghy 1.3.3.