ctapipe Daily Digest — Activity since 2025-10-21

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

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

  Nothing happening

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

  • Fix muon normalization pixel shapes #2865 [burmist-git]

    related to : #2845

    • mexanick

  • Fix gain selection of broken pixels, add tests for all extractors with CameraCalibrator #2867 [maxnoe]

    • burmist-git: Looks good for me.

    • burmist-git

    • burmist-git

    • burmist-git

    • TjarkMiener: Thanks! LGTM!

    • mexanick

      • mexanick: why is it 7? do we have 7 events in the test file? Shall we control it with max_events just in...

      • mexanick: please do not call the variable "array", I almost broke my head confusing it with a python...

  • Phi0 should be subtracted from phi not added. #2869 [burmist-git]

    Phi0 gives us the azimuth of the impact point, whereas before it was giving us the complementary angle of the impact point.

  • Visualizing schematic representation of Hillas parameters #2868 [vuillaut]

    This is an addition to the documentation to display as a schema what are the Hillas parameters:

    • maxnoe: Nice! Does it also work in TelescopeFrame? As we are about to drop support for image parameters...

    • sfegan: The way you have drawn the length and width is confusing since it shows them as a "diameter"...

  • Improve statistics aggregators #2848 [mexanick]

    Relax requirements on the data shape (aggregate arbitrary arrays on axis 0) Add option to aggregate statistics by...

    • mexanick: @maxnoe @TjarkMiener @matteobalbo the code has been updated implementing the chunking strategies...

    • maxnoe: Suggested change ...

    • maxnoe: Do we want to return the table here? Or just start index / stop index for indexing into the...

      • maxnoe: Especially the copy vs. view part should be part of the interface I think.

      • mexanick: good catch, you got me head scratching. SizeChunking returns views, but TimeChunking returns...

    • maxnoe: Do we only plan to use this with events? Or might it also be used for a monitoring table of...

      • mexanick: So far it is geared towards events, it requires event_id columns. While it is of course possible...

      • mexanick: I think, for generic table with other columns, the aggregation can be accommodated using the...

      • maxnoe: Why does it need the event_id column? It just splits by len, right?

    • maxnoe: Why do we raise an error here instead of yielding a single chunk?

      • maxnoe: I feel like we should add some configuration options here on the behavior for the case of a)...

      • mexanick: If the chunk size is set to None, a complete table will be used (i.e. user doesn't require...

    • maxnoe: Suggested change if...

      • mexanick: Is it needed? Does astropy allow negative time? It is AstroQuantity with the physical type of time.

      • maxnoe: Sure: In [3]: -5 * u.s Out[3]: <Quantity -5. s>

    • maxnoe: I think this loop here can be much simplified by computing the chunk once by doing: chunk_index...

      • mexanick: oh, I didn't think that python supports integer division for floats!

created at 2025-10-22 06:35 by dinghy 1.4.1.