Calculate and store true_disp per-telescope when running ctapipe-process#3003
Calculate and store true_disp per-telescope when running ctapipe-process#3003Copilot wants to merge 10 commits into
Conversation
|
For reference this is a test. There is an ongoing discussion within CTAO about whether github copilot or codex represents better value for money for the observatory for agentic coding. One of the key issues is whether github integration such as this is worth double the cost. Let's see how well copilot does. |
- Add TrueDispContainer class to containers.py with norm and sign fields - Add true_disp field to SimulatedCameraContainer - Calculate true_disp in ImageProcessor._calculate_true_disp when use_telescope_frame=True - Write true_disp to simulation/event/telescope/parameters in DataWriter - Update HDF5EventSource to read back true_disp fields - Add tests for the new functionality in test_image_processor.py" Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/63cfb286-711f-4644-aa96-b0acb650a6a5 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
…eview - Remove duplicate test functions in test_image_processor.py - Remove unused imports in test file - Update TrueDispContainer.sign field description to mention sign can be 0 Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/63cfb286-711f-4644-aa96-b0acb650a6a5 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
…las coords Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/63cfb286-711f-4644-aa96-b0acb650a6a5 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
…mula comment Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/63cfb286-711f-4644-aa96-b0acb650a6a5 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
|
@copilot Use existing test files available within the ctapipe framework to evaluate this work. Do not attempt to connect to the DESY cluster |
…/event The new true_disp tests now use a synthetic rectangular camera geometry and synthetic event data that can be run without any network access. This eliminates the need to connect to the DESY cluster during testing. Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/140fa9a8-8f5f-4f16-ae78-93155d5455f3 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
The three new tests ( Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Address failing checks |
Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/4532ae0b-e23a-439e-bca2-2827f5e42ffc Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
|
@copilot CI/lint still failing |
Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/68dff1ba-6558-446e-a9f4-f9bbd05c4772 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
Fixed in commit d6607a8 — Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Continue to address failing checks |
…oating point values Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/8d9a62d3-f9d7-4fc0-b9a9-a11f49e60d38 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
Fixed in commit 57ddb1c. The test failure was in The docs build failure (8 warnings from |
|
@copilot Address failing docs checks and ensure this PR contains adequate documentation |
… data format docs for TrueDispContainer Agent-Logs-Url: https://github.com/cta-observatory/ctapipe/sessions/128715bf-16e5-4cb4-843d-c258647fd041 Co-authored-by: STSpencer <31512502+STSpencer@users.noreply.github.com>
Addressed in commit 5c53271:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Address failing checks |
The latest CI run on commit 5c53271 shows docs ✅ and lint ✅ both passing. The single failing job ( |
TrueDispContainerclass tocontainers.pywithnormandsignfields (prefixtrue_disp)true_dispfield toSimulatedCameraContainerincontainers.pytrue_disp_normandtrue_disp_signinimage_processor.pywhenuse_telescope_frame=Trueand simulation data is availabletrue_dispcontainer tosimulation/event/telescope/parametersindatawriter.pyhdf5eventsource.pyto read backtrue_dispfields when loading existing HDF5 filestest_true_disp_calculation: verifies norm ≥ 0 and sign ∈ {-1, 0, 1}test_true_disp_requires_telescope_frame: verifies NaN defaults whenuse_telescope_frame=Falsetest_true_disp_no_simulation: verifies graceful handling of missing shower infodocs/changes/3003.feature.rstchangelog entryAltAzcoordinatestraitlets.traitlets.Kandtraitlets.traitlets.Vtonitpick_ignoreindocs/conf.pyto suppress pre-existing upstream warningsdocs/user-guide/data_format/index.rstto documentTrueDispContainerin the simulation parameters table entry