rest-for-physics / rest-for-physics/framework

TRestAnalysisPlot simplification using RDataFrame

Open
#3 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C++
Stars
19
Forks
14
PR merge metrics
No merged PRs in 30d

Description

A lot of efforts have been put so that TRestAnalysisPlot works efficiently opening thousands of files at once without major leak problems, and efficiently. So, right now, TRestAnalysisPlot is quite robust and efficient.

However, rewriting TRestAnalysisPlot using RDataFrame could strongly simplify/reduce the code.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the TRestAnalysisPlot implementation and its current file-opening and resource-management logic. Read the relevant ROOT RDataFrame documentation and compare it with the existing class behavior. Done means a substantially simpler implementation that retains the current efficiency and robustness when processing thousands of files.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
data-visualization
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.