Skip to content

Various updates and fixes#24

Merged
sawenzel merged 3 commits intoAliceO2Group:masterfrom
benedikt-voelkel:add-replay
Nov 9, 2021
Merged

Various updates and fixes#24
sawenzel merged 3 commits intoAliceO2Group:masterfrom
benedikt-voelkel:add-replay

Conversation

@benedikt-voelkel
Copy link
Contributor

  • store TGeoManager::GetPath() in MCStepLogger if track enters to
    restore geometry in replay. This is necessary in particular in case of
    steps on or in the vincinity of boundaries. Floating point precision
    might not be enough to find a node based on spatial coordinates. In
    addition, restoring from the path makes navigation and node finding
    obsolete during replay

  • extract TGEoVolume ID to VMC volume ID mapping. These are not the same
    for Geant4_VMC hence it is necessary. Mapping is required in replay,
    otheriwse, hit processing can fail.
    Mapping during replay is skipped if it is the identity.

  • make replay work for Geant4_VMC. This works thanks to above additions.

  • add possibility to load a custom user keepStep function from a ROOT
    macro

  • additional small fixes

* store TGeoManager::GetPath() in MCStepLogger if track enters to
  restore geometry in replay. This is necessary in particular in case of
  steps on or in the vincinity of boundaries. Floating point precision
  might not be enough to find a node based on spatial coordinates. In
  addition, restoring from the path makes navigation and node finding
  obsolete during replay

* extract TGEoVolume ID to VMC volume ID mapping. These are not the same
  for Geant4_VMC hence it is necessary. Mapping is required in replay,
  otheriwse, hit processing can fail.
  Mapping during replay is skipped if it is the identity.

* make replay work for Geant4_VMC. This works thanks to above additions.

* add possibility to load a custom user keepStep function from a ROOT
  macro

* update MCReplay/README.md

* additional small fixes
@sawenzel sawenzel merged commit 99c3e92 into AliceO2Group:master Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants