Deprecate Cluster::nhits field
Open
- Dominant language
- C++
- Stars
- 4
- Forks
- 7
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 3
Description
https://github.com/eic/EDM4eic/blob/f0805abdda5ccf5408f2eb7d415e6709c8f68f90/edm4eic.yaml#L286
This is redundant. We should just check consistency between lengths of the `hitContributions` vector and `hits` OneToMany relation.
Contributor guide
No contributing guide indexed for this repository
Research direction
Inspect edm4eic.yaml at line 286 and trace how Cluster::nhits, hitContributions, and the hits OneToMany relation are represented. Check the surrounding data-model definitions and any generated outputs or validation behavior. Done means the redundant field is deprecated and consistency is based on the two collection lengths.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100