ctapipe Bi-Weekly Digest — Activity since 2025-06-27

» cta-observatory/ctapipe | cta-observatory/ctapipe

• cta-observatory/ctapipe repo issues [back to top]

  • Generalise DL2 tables loading and filtering #2790 [Voutsi] enhancement

    Please describe the use case that requires this feature. Telescope cross-calibration. Describe the solution you'd...

  • IRF outputs for protons + electron A_eff need better HDUCLAS headers #2792 [kosack] bug

    Describe the bug The proton and electron effective areas in the IRF output have the same format as the gamma-ray...

    • maxnoe: There is no mechanism in GADF to do this, so for now I would propose to strip the GADF related...

  • Cut optimizer error #2765 [rcervinoucm] bug

    Hello, I am trying to use the ctapipe-optimize-event-selection tool as a previous step to create IRFs, and I am...

    • rcervinoucm: Thanks a lot for the help, it was finally an error from my side regarding the keys order.

  • `ctapipe-quickstart` prints many warnings #2793 [Tobychev] bug

    Describe the bug ctapipe-quickstart prints many warnings To Reproduce Steps to reproduce the behavior: >...

  • position_to_pix_index wrong behaviour for x,y outside camera #2795 [moralejo]

    ctapipe/src/ctapipe/instrument/camera/geometry.py Line 941 in 2bf1942 ...

    • maxnoe: #1828 (comment)

    • maxnoe: Intended, but not properly documented

    • maxnoe: This is the intended behavior. The returned value is int64.min to have a clear invalid value...

  • ctapipe-quickstart prints many warnings #2794 [Tobychev] bug

    Describe the bug ctapipe-quickstart prints many warnings To Reproduce Run ctapipe-quickstart. Expected behavior That...

    • kosack: Ah, yes that is just because the provenance add_input() now tries to get reference metadata, but...

    • Tobychev: Sure, or else the quickstart, who is the one making these filese, could maybe provide the...

  • templates for ctapipe-quickstart should not be the files used for testing #2796 [kosack] bug

    Describe the bug ctapipe-quickstart is meant to give the user a nice starting point to do an analysis, but right now...

• cta-observatory/ctapipe repo pull_requests [back to top]

  • Add possibility to produce DL3 with ctapipe #2727 [mdebony]

    The purpose of this PR is to add support for the creation of DL3 file in ctapipe. The current output format is the...

  • Add DL0 telescope event type, add missing types #2769 [maxnoe]

    Marking as draft for now, I will prepare a change request to the R1 model so that the additional types we define...

    • kosack

  • Add a component to reduce waveform readout window #2780 [maxnoe]

    The question is now where to best apply this. I see a couple of options: Add directly to ctapipe-process, this...

    • mdpunch: Still seems fine! (Next step, to add a "GRB HaST" component in another pull request?)

    • maxnoe: (Next step, to add a "GRB HaST" component in another pull request?) We don't simulate dead time...

    • mdpunch: (Next step, to add a "GRB HaST" component in another pull request?) We don't simulate dead time...

  • Fix warnings about missing meta in quickstart tool, fixes #2794 #2797 [maxnoe] no-changelog-needed

    • kosack

  • Modify camera monitoring data format #2787 [TjarkMiener] data model change

    This PR implements the camera monitoring data format, which we will be proposing next week at the DPPS workshop....

  • Add full cut optimization as introduced in pyirf 0.13 #2789 [LukasBeiske]

    This changes the PointSourceSensitivityOptimizer to use the full cut optimization introduced in pyirf 0.13. The...

    • maxnoe: you can just index by extname hdul["RAD_MAX"] and also check "RAD_MAX" in hdul

      • maxnoe: do not rely on indices

    • LukasBeiske: There was still an error with the application of the cuts in the irf tool. It should be correct...

    • maxnoe: If I remember correctly, these percentiles cannot be compared directly, as the eventdisplay-like...

    • LukasBeiske: If I remember correctly, these percentiles cannot be compared directly, as the eventdisplay-like...

    • LukasBeiske: Running both optimizations with finer grids: PointSourceSensitivityOptimizer: ...

    • kosack: gh_cut_efficiency_step=0.02 theta_cut_efficiency_step=0.02 If a smaller step is important to...

    • LukasBeiske: gh_cut_efficiency_step=0.02 theta_cut_efficiency_step=0.02 If a smaller step is important to...

  • Generalise table preprocessing #2791 [Voutsi] module:irf

    Closes #2790 Refactoring the DL2 processing for producing IRFs. The motivation is to use the same code for more...

    • mexanick: Overall looks good, however I'm wondering if this preprocessing module should go under...

    • LukasBeiske: Besides that, looks good to me! Kind of related #2789

      • LukasBeiske: This does not really override the column renaming, but extend it, no?

        • Voutsi: Yes. The default behaviour is to rename the same columns, the same way as before. The user can...

      • LukasBeiske: Is this intentional? If yes, why?

        • Voutsi: No it wasn't intentional. Thanks for spotting it.

      • LukasBeiske: These last 4 columns are missing the descriptions.

      • LukasBeiske: You can use double backticks to get sphinx to use mono font without a reference (e.g....

    • LukasBeiske: Regarding my above thread, this only extends the default renaming and does not overwrite it,...

      • Voutsi: I see, so you prefer not to extend it. I think you are right, that makes more sense. I will...

    • maxnoe: You added two changelog snippets here that contain text that is not really fitting for a...

    • maxnoe: Do we need a "disable_column_renaming" option if the default mapping is an empty dict (i.e. no...

    • maxnoe: What is the purpose of this option? From the help it's not clear to me.

      • Voutsi: These are the columns we want to keep from the input table without any renaming. I modified a...

    • maxnoe: This is not really an optimization (think performance improvement without API change). More a...

    • maxnoe: Why did you change to Any from the correct Dict[str, str]?

      • maxnoe: If the problem was the None, the correct way is to specify allow_none=True.

      • maxnoe: I also do not understand why we need the None case. Can't we keep the default but allow to set...

      • maxnoe: Ah, I see, the default depends on other traits. Traitlets has a mechanism for this. Define a...

  • Fix muon docstring #2798 [maxnoe] no-changelog-needed documentation-only

    • kosack

    • moralejo

created at 2025-07-11 10:53 by dinghy 1.4.1.