Edward Moyse authored 5 years ago and Walter Lampl committed 5 years ago

Ongoing effort to get Muon standalone working. Now added ability to set arguments, to be used in ART tests. Found some missing configuration which is now added. And a protection against a rare crash.

Fixes for configutation of MooSegmentCombinationFinder to allow it to be overridden properly for NCB
Added missing MuonChamberHoleRecoveryToolCfg for track building
General fixes to make the concurrency configurable at runtime
Added some args to allow new threaded tests to work
Whitespace fixes, and also a protection against crashing for ATLASRECTS-5087 (real fix still under investigation).

- Removed deprecated config flag
- Moved common tool configuration into new MuonRecToolsConfig.py (and removed this from MuonSegmentFindingConfig.py)
- Added a lot of new configuration to MuonTrackBuildingConfig.py
- Added TrackBuilding to MuonReconstructionConfig.py

William Axel Leight authored 5 years ago and Frank Winklmeier committed 5 years ago

The service now uses the MdtCondDbData object created by the MdtCondDbAlg, rather than the MdtCondSummarySvc.
The MdtIntersectGeometry objects are no longer cached in the service but created on the fly as needed, making the service much simpler.
Mutable data members are removed or made local variables.
The MdtCondDbData object and the MuonDetectorManager must now be passed in from outside: the clients of the service are adjusted accordingly.
The MdtCondDbAlg must now run during muon reconstruction: the configuration is updated accordingly.

Edward Moyse authored 5 years ago and Frank Winklmeier committed 5 years ago

- Removed deprecated config flag
- Moved common tool configuration into new MuonRecToolsConfig.py (and removed this from MuonSegmentFindingConfig.py)
- Added a lot of new configuration to MuonTrackBuildingConfig.py
- Added TrackBuilding to MuonReconstructionConfig.py