HGTD hits time compatibility check

Added a time compatibility check (algorithm TrackQualityCheckAlg ) among all the hits per track in HGTD, put the handles to all the decorations in TrackQualityCheckAlg . @aleopold

added HGTD Reconstruction master labels

assigned to @vraskina

added 1 commit

• 5d35666e - moved bitfield skeleton to TimeCompatibilityCheckAlg

Compare with previous version

Hi @vraskina & @aleopold, as I said we plan to build a new release soon (probably early next week), so it would be good to get this merged before that :)

mentioned in merge request !55837 (merged)

added 3 commits

• a7622d47 - changed decos naming
• 6787cf43 - followed up the review for MR
• 2d432756 - added the check for the last hit in ITk

Compare with previous version

Note that this one now has conflicts.

added 1393 commits

• 2d432756...0b1b56f4 - 1392 commits from branch atlas:master
• e9a2713e - resolved conflicts

Compare with previous version

added 1 commit

• e80cec29 - added holes search in ITk

Compare with previous version

added 1 commit

• c64ac4b4 - added a tool for the holes on track search in ITk

Compare with previous version

added 2 commits

• 4c3dabf5 - Revert "added a tool for the holes on track search in ITk"
• aff07520 - Revert "added holes search in ITk"

Compare with previous version

added 642 commits

• aff07520...73711512 - 640 commits from branch atlas:master
• a8b9f0e8 - ahite space change
• d2eba70a - Merge remote-tracking branch 'upstream/master' into HGTD_reco_track_summary

Compare with previous version

HI @vraskina, looks like your fork is set to Private, could you set it to Internal instead ? So that it does not appear on shifter's console. David as LVL2 gitmerge shifter

added 1691 commits

• d2eba70a...157a8011 - 1690 commits from branch atlas:master
• db7c3365 - Merge remote-tracking branch 'upstream/master' into HGTD_reco_track_summary

Compare with previous version

added 626 commits

• db7c3365...70a55d3a - 624 commits from branch atlas:master
• b351db6a - added decoration for the truth info of the time compatible hits
• 70823bcf - Merge remote-tracking branch 'upstream/master' into HGTD_reco_track_summary

Compare with previous version

added 835 commits

• 70823bcf...21850b02 - 832 commits from branch atlas:master
• 03e3da48 - added hit truth info to the decorations; changed the name of algo to be more general
• f1f1128e - Merge remote-tracking branch 'upstream/master' into HGTD_reco_track_summary
• 83297af2 - commented the .py files, so the modifications in TrackQualityCheckAlg are not...

Compare with previous version

added 1 commit

• 5c8043c7 - follow up on the review comments

Compare with previous version

added 1 commit

• 5c87a82a - removed the TrackQualityCheckAlg from python scripts

Compare with previous version

marked this merge request as ready

resolved all threads

This merge request affects 1 package:

• HighGranularityTimingDetector/HGTD_Reconstruction/HGTD_RecAlgs

This merge request affects 3 files:

• HighGranularityTimingDetector/HGTD_Reconstruction/HGTD_RecAlgs/HGTD_RecAlgs/TrackQualityCheckAlg.h
• HighGranularityTimingDetector/HGTD_Reconstruction/HGTD_RecAlgs/src/TrackQualityCheckAlg.cxx
• HighGranularityTimingDetector/HGTD_Reconstruction/HGTD_RecAlgs/src/components/HGTD_RecAlgs_entries.cxx

Adding @goetz ,@dshope ,@cohm ,@lshan as watchers

added review-pending-level-1 label

changed the description

CI Result SUCCESS (hash 5c87a82a)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 61871]

added review-approved label and removed review-pending-level-1 label

mentioned in commit 85f047fc

merged

added sweep:ignore label