Small modernization creation of a library in TAlignment
The goal of the library is to host AlignAlgorithm and its dependencies so that it can be used in python via cppyy. This will avoid going through the Detector Store for the output of the alignment process, which is not a proper solution and not even possible at all with DD4hep.
If you wonder why it's not "proper", this is because accessing the Detector store requires in principle to now which conditions to load, requiring in turn to know which event we are dealing with. Now at the end of the alignment process (finalize stage), we have no event and thus no access to the DetectorStore. Current implementation abuses the system by still going there and using "what is left", in a very non thread safe mode. But DD4hep's implementation of the interface is safe and thus won't let you access without a proper event.
Merge request reports
Activity
added 1 commit
- baa759b5 - Made library rather than module for TAlignment so that cppyy can access objects
mentioned in merge request !152 (merged)
assigned to @peilian
assigned to @sponce
- [2020-11-06 15:54] Validation started with lhcb-master-mr#1536
- [2020-11-06 20:18] Validation started with lhcb-master-mr#1537
- [2020-11-08 00:11] Validation started with lhcb-head-2#61
Edited by Software for LHCbadded 5 commits
-
baa759b5...0c39d5f1 - 2 commits from branch
master
- dff7dd58 - modernization of AlignAlgorithm (very partial)
- b549df20 - Modernization of WriteAlignmentConditions
- f645eda8 - Made library rather than module for TAlignment so that cppyy can access objects
Toggle commit list-
baa759b5...0c39d5f1 - 2 commits from branch
added lhcb-head-2 modernisation labels
mentioned in commit f1420237
mentioned in issue Moore#221 (closed)