eic / eic/epic

Detector ID constants use ECal*, HCal* etc, but all else uses Ecal*, Hcal*

Open
#557 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
31
Forks
61
Avg merge
3d 12h
Merged PRs (30d)
16

Description

### Environment: (where does this bug occur, have you tried other environments)
- Which branch (`main` for latest released): main
- Which revision (`HEAD` for the most recent): HEAD
- Any specific OS or system where the issue occurs? no
- Any special versions of ROOT or Geant4? no

### Steps to reproduce: (give a step by step account of how to trigger the bug)
The difference in capitalization makes this error-prone, e.g. https://github.com/eic/EICrecon/pull/1070.

### Expected Result: (what do you expect when you execute the steps above)
We should use a consistent capitalization.

### Actual Result: (what do you get when you execute the steps above)
We aren't using a consistent capitalization.

### Additional Context
Probably this can all be included in a major renaming overhaul...

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.