Software Meeting Minutes
---

# 201124: Software Meeting

Agenda: https://indico.cern.ch/event/978592/

Connected: Andre, Placido, Valentin,
Present:

Apologies: Ulrike

## MC Generators

* TODO: add production config files to whizard2 sindarin repo

## Simulation

* Issue with simulation of muons with fixed pT?
    * observed different duplicate vs phi using different muon simulation runs
    * Maybe different input (not pt, but energy)?
    * Different Theta range, 0-90 degrees before, 0-180 degrees now

## Reconstruction

### Tracking

### Particle Flow

### Digitizer

### Event Display

### FlavourTagging

### Long-Lived Particle

### dE/dX

### LCGeo Test

## Infrastructure

## iLCDirac

* New version coming ASAP
    * DiracOS v1r11 tested (should test v1r16 by now...)
* Calice moving from LCG FileCatalog to DFC, keep v6r22 with lcg bundles, because those contain LFC bindings still

## key4hep

### Date releases

#### 2020-11-16

* Issues with renaming of packages
* Gaudi linking problems

### Gaudi

* cmake-modernization PR merged: https://gitlab.cern.ch/gaudi/Gaudi/-/merge_requests/986#note_3895786
* tried it out with k4fwcore already: https://github.com/vvolkl/K4FWCore/tree/cmake-modernization
    - working so far, but no K4FWCoreConfig.cmake generated! (policy is for users to generate their own )
* Finalizing PR
    * new pre-release version  v01-00pre02
* Writing guide for upgrading to new gaudi cmake
    *

### CI

* https://cern.service-now.com/service-portal?id=ticket&table=u_request_fulfillment&n=RQF1650416
* iLCSoft deployment: https://gitlab.cern.ch/CLICdp/SoftwareConfigurations/iLCSoft

* Working on incremental builds and deployments
    * Will be controlled from gitlab
* https://gitlab.cern.ch/key4hep/k4-deploy

* spack test -> recursion error
    * https://github.com/key4hep/k4-spack/issues/89

* Integrate with CDash, post updated packages

* Good to test spack HEAD

* Using dev-build for CI builds
    * Disable building of other packages?
        * set 'everything' to `buildable=false`
    * Tried it in
        * https://github.com/key4hep/k4FWCore/blob/master/.github/workflows/test.yml
        * https://github.com/key4hep/EDM4hep/pull/96
    * seems to work well, but issue with gaudi tests:
        * need runtime environment for upstream components
        * should set it up via env script, but this works only for the repository itself and gaudi, not other modules it depends on
* github actions:
    * use CVMFS setup action from Wouter
        * tried it out in https://github.com/HEP-FCC/k4Gen/blob/main/.github/workflows/test.yaml
        * seems to work well, can include it in other repos
    * build docker container that also contains (the k4fork of) spack and key4hep-spack
        * triggered when either is updated
    * action template for all k4Packages
* Refactor github actions
    * github actions template repository to configure compiler etc in one place for all packages

### Documentation website

* Create in cern gitlab, push to openstack/webEOS and redirect there from key4hep.github.io
    * TODO: get sftkey4hep account from Enric

#### Environment Script

#### View

* WIP, also Thomas Madlener looking into it
* Creating a view is just one spack command
    * `spack view [...]`

### key4hep-spack (renamed)

* updated key4hep-stack bundle package
    * PR: https://github.com/key4hep/k4-spack/pulls
* spack namespace: k4 -> key4hep ?
    * left it as is

### LCFIPlus inside Gaudi

* Valentin looking into it because of interest from FCC side

### podio

### EDM4hep

* Running Gean4EDM4hep output failing for dev3 now
    * https://travis-ci.com/github/key4hep/EDM4hep/jobs/433677959#L870
    * Plugin moving to DD4hep, so low priority
* Update doxygen generation

### k4FWCore

* Fixed user-written TFile issue: https://github.com/key4hep/k4FWCore/pull/23
    * added bunch of tests
    * still need to update the spack recipe with test dependencies

### k4MarlinWrapper (renamed)

* Standalone converter running, to be integrated with the wrapper
* Current status: https://github.com/fdplacido/lcio2edm4hep_converter/blob/master/k4TestFWCore/src/components/EmptyAlg.cpp

#### LCIO <-> EDM4hep conversions

* Conversion service from Gaudi
    * under investigation by Valentin
    * At the moment only used for geometry conversion
* How to do this most generically?
* How to tell which collections to import and export for given wrapper processor?

### spack

* PR in spack: root recipe overhaul
    * https://github.com/spack/spack/pull/19642
    * needs testing
* Fixed a bug with roofit variant
* Update plan?

Opened PRs for a bunch of improvements:
* https://github.com/spack/spack/pull/20064
* https://github.com/spack/spack/pull/20065
* https://github.com/spack/spack/pull/20030

#### spack build-dev

* E.g.: building edm4hep on local podio
    * add podio with buildable=false and the path to the packages.yaml
    * or dev-build podio and then use that "hash" for dev-build edm4hep
* Tried this out, works well, especially with ccache enabled
    * only concretizing is slow
    * when recompiling many times, better to use spack build-env or `spack load` all dependencies
    * `spack develop`: create an "environment" with multiple "build devs"

## FCCee Detector Model

## SCT Integration

* WIP: Add TPC xml as "Aurora" Package
*

## FCCSW and key4hep

* Adapted to EDM4hep
* Spack install and GDML
    * Fails to validate the GDML with the spack release
    * Works with LCG installation
* Splitting FCCSW into separate repositories
    * finished already: hep-fcc/k4Gen
    * under development: key4hep/k4SimDelphes
    * underway: hep-fcc/k4SimGeant4
    * underway: hep-fcc/k4RecTracker
    * underway: hep-fcc/k4RecCalorimeter
* Used for spack build
    * Three failing tests, using the same HepMC file

## Snowmass

## AOB

### Next Meeting

December 8, 2020, 13:00