rest-for-physics / rest-for-physics/framework
Add a mechanism to create process dependencies
Nobody has claimed this yet.
- 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
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
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