eic / eic/EICrecon

HcalEndcapPInsertClusters warning: Proto-cluster has highest energy in CellID 1168810095, but no mc hit with that CellID was found.

Open
#321 0 comments 0 reactions 0 assignees View on GitHub
bug topic: calorimetry
Dominant language
C++
Stars
13
Forks
42
Avg merge
18h 44m
Merged PRs (30d)
55

Description

### Environment: (where does this bug occur, have you tried other environments)
- Which branch (often `main` for latest released): main
- Which version (or `HEAD` for the most recent on git): 0.3.4
- Any specific OS or system where the issue occurs? eic-shell

### Steps to reproduce: (give a step by step account of how to trigger the bug)
1. Full production job, brycecanyon, `S3/eictest/EPIC/EVGEN/DIS/NC/18x275/minQ2=1000/pythia8NCDIS_18x275_minQ2=1000_beamEffects_xAngle=-0.025_hiDiv_vtxfix_1_007.hepmc.gz`
2. Reconstruction with `run_eicrecon_reco_flags.py`

### Expected Result: (what do you expect when you execute the steps above)
No warnings should be occur.

### Actual Result: (what do you get when you execute the steps above)
Many warnings like:
```
[2022-11-08 07:00:19.542] [c] [warning] Proto-cluster has highest energy in CellID 1168810095, but no mc hit with that CellID was found.
```

### Additional context
1. We should print ou the CellID as hex, since it is a bit pattern, i.e. format `%x`.
2. At some point we changed in juggler how we handle merged hits to avoid impossible masked CellID values. Maybe this is a symptom?

Contributor guide

Open the contributing guide

Research direction

Reproduce the warning with the listed brycecanyon input and run_eicrecon_reco_flags.py in eic-shell. Trace HcalEndcapPInsertClusters from the warning text and inspect how merged hits and CellID values are handled. Done means the reconstruction no longer emits the unexpected warnings, with CellID output using hexadecimal formatting where applicable.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.