Dominik Smith authored 10 months ago and Sergey Gorbunov committed 9 months ago

- Removed global ca::TrackFit instance from ca::TripletConstructor.
- Removed access to ca::InputData from ca::TripletConstructor (all needed data is already in ca::WindowData).
- Switched from global to window-based hit index in ca::Branch. Allows simplification of many interfaces
  (removal of ca::InputData).
- Cleaned up ca::TrackFinder.
- In ca::TrackFinder, replaced separate vector instances which store
  upper and lower bounds of windows by single vectors that store
  these bounds together as a pair.
- Cleaned up ca::TrackFinderWindow.

Ca: make CaTrackFit work with different data types: SIMD and non-SIMD tracks, single and double precision

Pierre-Alain Loizeau authored 11 months ago and Pierre-Alain Loizeau committed 9 months ago

- Switch some TOF mapping Init logs to debug in CbmL1

cbm::algo::ca::TrackParamBase (and derivatives) are moved to the KfCore library (now, cbm::algo::kf::TrackParamBase)

Additionally, dependiencies are cleaned-up

- Cleaned up ca::TripletConstructor.
- Removed fCurrentIterationIndex and access to it from ca::WindowData.

Dominik Smith authored 11 months ago and Florian Uhlig committed 10 months ago

- Removed access to ca::Framework from ca::TrackExtender, ca::TrackFitter
  ca::TripletConstructor, ca::CloneMerger, ca::TrackFinder and ca::TrackFinderWindow.
- Simplified the interfaces of these classes.
- General instruction-wise cleanup of these classes (applied std functions in
  some places etc.). In particular, streamlining of ca::TrackFinder.
- Made debugging functions in ca::Framework static.
- Moved TrackingMode enum from CaFramework.h to CaParameters.h (needed
  to avoid cyclic header file dependency).
- Moved some magnetic-field dependent variables to ca::TripletConstructor (they are
  only used there).
- Restricted the scope of many variables, i.e. turned class-fields into
  local variables in many instances (e.g. fvMonitorDataThread, fvTrackFinderWindow,
  fTrackingMode, fvWData, fvWindowStartThread, fvWindowEndThread,
  fvStatNwindows, fvStatNhitsProcessed, fvHitFirstTriplet and fvHitNofTriplets).
- Eliminated the Init() functions of some classes. Moved initialization
  to the constructors.
- Improved distinction between class-parameters and working data.
  Class parameters are passed to constructor, working data is passed as
  arguments to function calls.

Sergei Zharko authored 1 year ago and Sergey Gorbunov committed 1 year ago

This commit introduces the configuration header TrackingChainConfig, which contains
a minimalistic set of parameters to control the TrackingChain execution scenarios.

Sergey Gorbunov authored 1 year ago and Florian Uhlig committed 1 year ago

the MR should fix the issue https://redmine.cbm.gsi.de/issues/3149

It contains the following fixes to the CA tracking:

1. bugfix in the Jacobian of the track extrapolation.
   It was introduced during the rewrite of the math
2. tune binning of the hit grid.
   A little decrease of the efficiency happened after the code rewrite
3. tune the cuts for newly introduced hit ranges.
   The new formulae were not tuned
4. revert triplet cut on space + time chi2. 
   After the split of space and time Chi2 in the track fit,
   the chi2 cut was changed w/o tuning. Replace it with the old cut.