Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BAD PR, NOT REBASED. Fix too many clusters in si pixel cluster producer for 808 #14665

Conversation

VinInn
Copy link
Contributor

@VinInn VinInn commented May 27, 2016

back port of #14662
require rebase.
if somebody knows how do it from here is welcome

hroskes and others added 30 commits April 11, 2016 21:07
… is smaller than or equal to the latest IOV, exit.
konec and others added 11 commits May 24, 2016 10:59
Now it is done if in case |eta|==115 and hits are ONLY in 3 layers out of (7,8,9,17)
It has no effect for MC shince hits with this quality are filtered out, but
is usefull for data-MC comparistion (if one remove the filter)
…ing-cleanup-80x

Clean-up L1T unpacker warnings
Update OMTF treatment of Muons at edge of OMTF acceptance
 Add a cut in the HLTPFJetIDProducer module.
…05-20-2300

Fix for runtestTqafTopEventSelection unit test: use againstElectronVLooseMVA6 instead of againstElectronVLooseMVA5
…kport

CTPPS: new DataFormats related to TOTEM RP raw data and digi - backport of PR cms-sw#13837
Fixes for RunInfo O2O and cleanup of PopCon interface (80X)
…-entry-hash-80X

Fix for GTEntry_t hash function
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @VinInn (Vincenzo Innocente) for CMSSW_8_0_8_patchX.

It involves the following packages:

Alignment/OfflineValidation
CondCore/CondDB
CondCore/PopCon
CondFormats/Common
CondTools/RunInfo
Configuration/Applications
Configuration/StandardSequences
DQM/HLTEvF
DQM/HcalCommon
DQM/HcalTasks
DQM/Integration
DQM/L1TMonitor
DQM/Physics
DQM/SiPixelMonitorClient
DQM/TrackingMonitorClient
DQMOffline/Configuration
DQMServices/Components
DQMServices/FileIO
DQMServices/StreamerIO
DataFormats/CTPPSReco
DataFormats/DetId
DataFormats/L1TMuon
DataFormats/TotemDigi
DataFormats/TotemRPDetId
EventFilter/L1TRawToDigi
EventFilter/Utilities
GeneratorInterface/Pythia8Interface
HLTrigger/JSONMonitoring
HLTrigger/JetMET
L1Trigger/CSCCommonTrigger
L1Trigger/CSCTriggerPrimitives
L1Trigger/Configuration
L1Trigger/L1TCalorimeter

@ghellwig, @covarell, @emeschi, @cerminar, @Martin-Grunewald, @rekovic, @vanbesien, @govoni, @perrotta, @civanch, @cmsbuild, @fwyzard, @davidlange6, @cvuosalo, @mdhildreth, @deguio, @slava77, @ggovi, @mmusich, @mulhearn, @perrozzi, @thuer, @dmitrijus, @franzoni, @mommsen can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @apfeiffer1, @barvic, @batinkov, @makortel, @GiacomoSguazzoni, @rappoccio, @rovere, @valuev, @agrohsje, @tocheng, @tlampen, @mschrode, @mmusich, @rociovilar, @threus, @dgulhan, @mkirsano this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.
Note that this branch is designed for requested bug fixes specific to the CMSSW_8_0_8 release.
If you wish to make a pull request for the CMSSW_8_0_X release cycle, please use the CMSSW_8_0_X branch instead

cms-bot commands are list here #13028

@VinInn VinInn changed the title Fix too many clusters in si pixel cluster producer for 808 BAD PR, NOT REBASED. Fix too many clusters in si pixel cluster producer for 808 May 27, 2016
@slava77
Copy link
Contributor

slava77 commented May 27, 2016

need to start from 808patch1

@davidlange6
Copy link
Contributor

To be clear, we need both a 80x and s 808patchx pr

Cheers,
David

On 27 May 2016, at 17:36, Slava Krutelyov <notifications@github.commailto:notifications@github.com> wrote:

need to start from 808patch1


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//pull/14665#issuecomment-222179245, or mute the threadhttps://github.com/notifications/unsubscribe/AEzywyIcrekNE64rsK_0cJ8EA6WHGeSUks5qFw9_gaJpZM4IonD9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment