Phase 1 training for BoostedDoubleSV tagger for AK8 jets

[ferencek]

New Phase 1 training for BoostedDoubleSV tagger for AK8 jets.

Request to add new training to externals is here cms-sw/cmsdist#2951

@cvernier

[cmsbuild]

A new Pull Request was created by @ferencek (Dinko Ferencek) for master.

It involves the following packages:

RecoBTag/SecondaryVertex

@perrotta, @cmsbuild, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@imarches, @acaudron, @JyothsnaKomaragiri, @mverzett, @ferencek, @pvmulder this is something you requested to watch as well.
@Muzaffar, @davidlange6, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

[ferencek]

Please hold off any tests until the external training file is integrated into 91X IBs.

[iahmad-khan]

please test with cms-sw/cmsdist#2953

[cmsbuild]

The tests are being triggered in jenkins.
Using externals from cms-sw/cmsdist#2953
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19125/console

[cmsbuild]

+1
Tested at: 7f2561c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18318/19125/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18318/19125/summary.html

Comparison Summary:

• You potentially added 23 lines to the logs
• Reco comparison results: 1633 differences found in the comparisons
• DQMHistoTests: Total files compared: 23
• DQMHistoTests: Total histograms compared: 1921491
• DQMHistoTests: Total failures: 36000
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1885318
• DQMHistoTests: Total skipped: 173
• DQMHistoTests: Total Missing objects: 0
• Checked 94 log files, 14 edm output root files, 23 DQM output files

[slava77]

what is the plan to have this in GT?

[ferencek]

At the moment there is no plan. From the past experience it is much easier, more convenient, and faster to handle training files as externals. Perhaps things are better now with GTs but I doubt we can get a new GT created on the same times scale on which externals can be integrated in the IBs.

[slava77]

@ferencek
what is the plan to add these taggers to DQM?

[slava77]

+1

for #18318 7f2561c

• changes are as described
• jenkins tests pass and comparisons with baseline show no differences (the double-boosted tagger is not monitored)

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @Muzaffar, @davidlange6, @smuzaffar

[slava77]

@davidlange6
just FYI, the external needed for this PR was merged to cms-sw:IB/CMSSW_9_1_X/gcc530 on Apr 13

[ferencek]

@slava77, at the moment DQM is implemented only for taggers run in the standard reco. And in the standard reco we run b tagging only on ak4 jets. At some point in the future this tagger could be included in DQM but for now there are no concrete plans that I am aware of.

[slava77]

On 4/18/17 7:04 AM, Dinko Ferencek wrote: @slava77 <https://github.com/slava77>, at the moment DQM is implemented only for taggers run in the standard reco. And in the standard reco we run b tagging only on ak4 jets. At some point in the future this tagger could be included in DQM but for now there are no concrete plans that I am aware of.

double-boosted is a part of standard processing (miniAOD step). So, it should be monitored in the corresponding miniAOD DQM/validation sequences.

…

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#18318 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEdcbhzMQIjTwaxf2viPyd2j38hXrNXIks5rxML3gaJpZM4M6nOo>.

[ferencek]

Thanks for pointing out the MiniAOD-specific DQM sequence. To be honest, it completely slipped my mind. We will then have to add this to the development plans.
@mverzett @JyothsnaKomaragiri @imarches @cvernier