Resolve "Add time clustering support"

Renat Dusaevrequested to merge
148-sadc-time-clusters into master

This merge request introduces time clustering support for ECAL DISABLED by default. It is needed prior to API transition defined in #155 (closed) to avoid messing up the history:

  • first this branch will be merged to master to add (disabled) time clustering support
  • then #155 (closed) will be merged with master to introduce API change

For rationale please expand:

New design is needed because:

  • it facilitates some important improvements we need
  • API change in #155 (closed) introduce faint changes which may have effect on the calibrations people are currently doing
  • current issue ( #148 ) shall continue on new design

However new design bears time clustering support already (this branched is merged there). So, if one will try to merge v2 branch to master, one should do either:

  • disabling time clustering in v2 to re-enable it in future. Note, that while merging master with this branch then, there will be conflicts.
  • introduce time clustering before time clustering is logically introduced. anyway this branch will become harder to merge.

Merge request reports