rest-for-physics / rest-for-physics/detectorlib
Add "Unique Name" field to readout modules
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 2
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Description
When the readout plane is large and contains many readout modules, the current implementation might be difficult to identify each. We have:
TRestDetectorReadoutModule::fModuleIDto identify module id. It is an int number, may not be so clear to the users.TRestDetectorReadoutModule::fModuleNameto identify the module type, either 128 channel MicroMegas or pixel readout module.
So I suggest to add a new field fUniqueName to TRestDetectorReadoutModule class. It would be a string data member, and therefore be better to record module names or serial numbers. For example, "V2M2", "V6_05", "A1580HN.46", something like that.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the TRestDetectorReadoutModule class and inspect how fModuleID and fModuleName are declared, persisted, and exposed. Add the requested fUniqueName string for user-defined names or serial numbers, then verify that readout modules can retain and report values such as "V2M2" or "A1580HN.46".
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100