key4hep / key4hep/k4MarlinWrapper

converter script: Parameterized processor names trip up converter script

Open
#93 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
3
Forks
19
PR merge metrics
No merged PRs in 30d

Description

Marlin seems to support the following construct (taken from [`CLICPerformance/fcceeConfig/fccReconstruction.xml`](https://github.com/iLCSoft/CLICPerformance/blob/master/fcceeConfig/fccReconstruction.xml)):

```xml


10




```

this gets converted into
```python
algList.append(MyDDCaloDigi_${CalorimeterIntegrationTimeWindow}ns)
```

which is an obvious syntax error in python.

Contributor guide

Open the contributing guide

Research direction

Start with the converter script that turns the shown Marlin processor declaration into an algList.append entry, using the XML example in the issue as the reproduction. Trace how processor names containing ${...} are emitted; done means the parameterized name no longer produces invalid Python while ordinary processor names still convert correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.