key4hep / key4hep/TrackerPerformanceScripts
Move ResVDX_UV_ from condorJobs_reco to config file?
- Dominant language
- Python
- Stars
- 2
- Forks
- 10
- PR merge metrics
- No merged PRs in 30d
Description
I know this is from Gaelle but on the one hand I am completely confused what is supposed to hapen here syntax wise, while on the other hand we should make the whole `ResVDX_UV_` optional (also below).
It would be nice to be able to specify additional arguments to k4run in the config and then this could go there.
_Originally posted by @Zehvogel in https://github.com/key4hep/k4DetPerformance/pull/3#discussion_r1634939994_
Contributor guide
Research direction
Start by locating condorJobs_reco and the existing k4run invocation, then read the linked pull-request discussion about ResVDX_UV_. Clarify the intended config syntax and optional behavior before changing anything; done means additional k4run arguments can be specified in configuration and ResVDX_UV_ is optional.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100