WIP: Add MC checking sequence

Adds MC checking sequence as implemented before in https://gitlab.cern.ch/lhcb/Brunel/blob/master/Rec/Brunel/python/upgrade_options/HLT1BaseLine.py#L222

More description to be added later.

Very nice! cc @dovombru

added 2 commits

• 3d8473fe - Add HistogramSvc to EverythingHandler
• fd5688eb - Write histograms

Compare with previous version

how far is this @sstahl? can we merge? i would like to get the merges going, otherwise the rebase mess is going to be unbearable when more MRs pop up

It works, but probably can be made a bit nicer.

resolved all discussions

shall we test this by running the new options file and checking for no warnings/errors/fatals?

btw, it's a bit strange that the options in Hlt1Conf import RecoConf which imports Hlt1Conf itself, but you probably realized this. We have to move things out of Hlt12Conf

assigned to @rmatev

added 1 commit

• caa4948b - Add copyright and move file to RecoConf

Compare with previous version

resolved all discussions

added 41 commits

• caa4948b...39ee1806 - 37 commits from branch master
• 95db7d45 - Add simple MC checking sequence
• cba16e5e - Add HistogramSvc to EverythingHandler
• 16717a15 - Write histograms
• 2ac8fae2 - Add copyright and move file to RecoConf

Compare with previous version

rebased on master

added 1 commit

• 2fe30057 - Do not rely on gaudi/Gaudi!929 (merged)

Compare with previous version

added 4 commits

• 5a8b2d73 - Add simple MC checking sequence
• d24d80d4 - Do not rely on gaudi/Gaudi!929 (merged)
• e329cee8 - Fix pytest discovery on new modules
• eb8c3e16 - Use TrackStateProvider.UnsafeClearCache from Rec!1584 (merged)

Compare with previous version

@nnolte @apearce I removed the dependency on gaudi/Gaudi!929 (merged) and added one on Rec!1584 (merged) (which we anyway had, but this time it can be merged). I also fixed the new magic modules for pytest and a silly issue in HLT1. I'm not quite sure how you ran them before

What's left is just to exercise the new code

added 1 commit

• 91f04d98 - Add qmtest for mc checking sequence

Compare with previous version

added 6 commits

• 84f579d1 - Do not rely on gaudi/Gaudi!929 (merged)
• 85f2c78b - Fix pytest discovery on new modules
• 9342d119 - Use TrackStateProvider.UnsafeClearCache from Rec!1584 (merged)
• 0f82b1e2 - Fix fallout from previous MRs
• ea09a286 - Add simple MC checking sequence
• 46d28f71 - Add qmtest for mc checking sequence

Compare with previous version

rebased on top of !189 (merged) . Probably needs to wait for a fix in Rec/master