Remove muon isolation filter for TkAlZMuMuHI and TkAlUpsilonMuMuHI. …

[jrcastle]

…Experience with 2015 data shows that these filters kill yields in heavy ions

[cmsbuild]

A new Pull Request was created by @jrcastle (James Castle) for CMSSW_8_0_X.

It involves the following packages:

Alignment/CommonAlignmentProducer

@diguida, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @pakhotin, @tocheng, @tlampen, @mschrode, @mmusich this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[jrcastle]

Hi,

For more information, when the old common alignment producer was run over the 2015 PbPb data, a total of 300 Z events survived the skim. Other analyses from heavy ions have shown yields of around 4000. Cuts were comparable between the two approaches, however the common alignment producer was using a muon isolation filter. Removal of this filter increased the number of Zs by a factor of 10. Since the Z yield for heavy ions is so low already, this change is implemented to get as many as possible for alignment efforts.

[tlampen]

This is a good finding!! -Tapio

On Fri, 29 Jan 2016, James Castle wrote:

Hi,

For more information, when the old common alignment producer was run over the
2015 PbPb data, a total of 300 Z events survived the skim. Other analyses from
heavy ions have shown yields of around 4000. Cuts were comparable between the
two approaches, however the common alignment producer was using a muon
isolation filter. Removal of this filter increased the number of Zs by a factor
of 10. Since the Z yield for heavy ions is so low already, this change is
implemented to get as many as possible for alignment efforts.

—
Reply to this email directly or view it on
GitHub.[AADIn0Dh1EIIEZkZJmKQHO-wIPNxN5Lkks5pe9kYgaJpZM4HPjZL.gif]

[mmusich]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10872/console

[cmsbuild]

+1
Tested at: 1331bbd
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13130/10872/summary.html

[mmusich]

+1

[cmsbuild]

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

[cmsbuild]

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

[davidlange6]

+1