key4hep / key4hep/k4MarlinWrapper
converter script: Parameterized processor names trip up converter script
- 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
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