rest-for-physics / rest-for-physics/framework
Migrate drawing and analysis methods to dedicated classes or namespaces
@juanangp is already working on this.
Since Apr 19, 2022.
- Dominant language
- C++
- Stars
- 19
- Forks
- 14
- PR merge metrics
- No merged PRs in 30d
Description
Migrate drawing and analysis methods within the different libraries to a namespace such as TRestXXXPainter and TRestXXXAnalysis:
TRestXXXPaintershould implement all the drawing methods within theXXXlibrary.TRestXXXAnalysisshould implement all the analysis methods within theXXXlibrary.
All the libraries inside the repository should be migrated to this schema. Perhaps, the implementation of TRestHitsPainter and TRestHitsAnalysis inside framework can be discussed. We should assing a developer to migrate the different libraries:
- rest-for-physics/detectorlib
- rest-for-physics/rawlib
- rest-for-physics/geant4lib @lobis
- rest-for-physics/tracklib @juanangp
- rest-for-physics/connectorslib
- rest-for-physics/framework ?
Ideally a template should be provided to use a similar schema for all the libraries by defining dedicated TRestXXXPainter and TRestXXXAnalysis namespaces in every single library.
We should discuss the need of other generic namespace such as TRestXXXTools
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.
Assessment
This issue has not been assessed yet.