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

Add a mechanism to create process dependencies

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

Nobody has claimed this yet.

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

Description

Sometimes a process will require that a process has been placed previously inside the data chain. Then, it would be interesting to introduce a mechanism to indicate those dependencies inside the process.

We could think of something like inserting a line of code as follows in our process:

// Values 1 or 2 define if the process is to be recommended or required. It could be an enum giving better legibility to the code.
ProcessRequired("TRestMyRecommendedProcess", 1);
ProcessRequired("TRestMyRequiredProcess", 2);

or

ProcessRecommended("TRestMyRecommendedProcess");
ProcessRequired("TRestMyRequiredProcess");

or

AddProcessDependency("TRestMyRecommendedProcess", 1);
AddProcessDependency("TRestMyRequiredProcess", 2);

Then, the processing will warn in case of a recommended process is not present, or prompt an error in case a required process is not available.

@rest-for-physics/core_dev

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

The issue names no files or tests. Start by tracing where processing checks process availability in the data chain, then clarify how recommended dependencies should warn and required dependencies should error. Done means the dependency API and both behaviors are agreed and specified.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Feature
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.