key4hep / key4hep/TrackerPerformanceScripts

Move ResVDX_UV_ from condorJobs_reco to config file?

Open
#8 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.