Add automated reconstruction quality analysis workflow for truth-reconstruction comparisons

[Copilot]

• Identify root cause of ValueError in reconstruction quality analysis workflow
• Fix awkward array boolean evaluation issue in compare_reconstruction_truth.py
• Replace problematic if not data: with proper type-specific checks
• Test script syntax and logic validation
• Validate fix works in CI environment

Fixed Issues

Problem: The GitHub Actions workflow reconstruction-quality-analysis was failing with:

ValueError: the truth value of an array whose length is not 1 is ambiguous; use ak.any() or ak.all()

Root Cause: Line 278 in scripts/compare_reconstruction_truth.py was using if not data: where data is an awkward array returned from tree.arrays(). Awkward arrays cannot be directly evaluated as boolean when they contain multiple elements.

Solution: Replaced the problematic boolean evaluation with proper type-specific checks:

• isinstance(data, dict) and not data - handles empty dict from error cases
• hasattr(data, 'fields') and len(data.fields) == 0 - handles awkward arrays with no fields

This maintains the same logical behavior while avoiding the ValueError with awkward arrays.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[github-actions]

Capybara summary for PR 2074

• rec_dis_10x100_minQ2=0_craterlake_10x100
• rec_dis_10x100_minQ2=1000_craterlake_tracking_only
• rec_dis_18x275_minQ2=0_craterlake_18x275
• rec_dis_18x275_minQ2=0_craterlake_18x275_MT
• rec_dis_18x275_minQ2=0_craterlake_18x275_ST_MT
• rec_dis_18x275_minQ2=1000_craterlake_18x275
• rec_dis_18x275_minQ2=1000_craterlake_18x275_MT
• rec_dis_18x275_minQ2=1000_craterlake_18x275_ST_MT
• rec_dis_5x41_minQ2=0_craterlake_5x41
• rec_e_1GeV_20GeV_craterlake
• rec_pi_1GeV_20GeV_craterlake
  ^{_{Last updated 2025-09-10T22:13-04:00 8df9a7b}}

[wdconinc]

@copilot The new job steps fail with:

Running reconstruction quality analysis for particle gun...
Traceback (most recent call last):
  File "/home/runner/work/EICrecon/EICrecon/scripts/compare_reconstruction_truth.py", line 356, in main
    result = analyze_file(file_path, file_type)
  File "/home/runner/work/EICrecon/EICrecon/scripts/compare_reconstruction_truth.py", line 278, in analyze_file
    if not data:
  File "/opt/hostedtoolcache/Python/3.10.18/x64/lib/python3.10/site-packages/awkward/highlevel.py", line 1752, in __bool__
    raise ValueError(
ValueError: the truth value of an array whose length is not 1 is ambiguous; use ak.any() or ak.all()
Analysis failed for gun file

Revisit the scripts/compare_reconstruction_truth.py script and test to ensure it succeeds, then commit changes to this PR.

[Copilot]

@copilot The new job steps fail with:

Running reconstruction quality analysis for particle gun...

...

Fixed the awkward array boolean evaluation error in commit 8df9a7b. The issue was on line 278 where if not data: was trying to evaluate an awkward array as boolean, which is ambiguous when the array has multiple elements. Replaced it with proper type-specific checks that handle both empty dict (error cases) and awkward arrays with no fields.