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
SiStripMonitorCluster & SiStripMonitorDigi - Fix the FED digi/cluster plots ranges and error [Backport for 10_1_X of #23307] #23308
Conversation
Phase2 Outer Tracker : Incorporating recent changes from TEDD Design + cleaning
….h in CMSSW_9_3_1)
This allows dropping the explicit --filein from upgrade HARVESTING. The parameter will be set in WorkFlowRunner based on the position of HARVESTING step. Setting it explicitly here basically fixes the RECO+DQM+VALIDATION step to be number 3 (which won't be the case for combined premixing stage1+stage2 workflow). Anyway HARVESTING,ALCA is the typical order in non-upgrade MC workflows, so this change makes them a tiny bit more consistent.
Fixing Default Values of EgammaHLTGsfTrackVarProducer to be only used if there is a track found : 102X
Temporary protection against missing jet corrector collection
…ucer Remove Parameter existence check since done by fillDescriptions
revert stream::producer to simple producer to avoid crashes in HTXSRivetProducer
Use GBRForest in PATMuonProducer
SiStripEventSummary fixes for special runs in local-DAQ
Magnetic Field: Remove Dependency on DDSolidShape
isolate TTree::Fill so IMT doesn't steal long running CMS tasks
Phase2-hgx107 Change the basic geometry of HGCal from FlatTrd to FlatHexagon
Strip hit eff update 10_2_x
A new Pull Request was created by @hbecerri for CMSSW_10_1_X. It involves the following packages: CalibCalorimetry/HcalAlgos The following packages do not have a category, yet: DQM/OuterTrackerL1 @lveldere, @ianna, @kpedro88, @fabozzi, @nsmith-, @rekovic, @thomreis, @alberto-sanchez, @vanbesien, @arizzi, @perrotta, @civanch, @monttj, @silviodonato, @cmsbuild, @fwyzard, @GurpreetSinghChahal, @davidlange6, @smuzaffar, @Dr15Jones, @efeyazgan, @mdhildreth, @jfernan2, @cerminar, @slava77, @ggovi, @fabiocos, @ssekmen, @Martin-Grunewald, @prebello, @vazzolini, @kmaeshima, @arunhep, @perrozzi, @dmitrijus, @alja, @franzoni, @gpetruc, @lpernie can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
This needs to be rebased so it only includes the commits actually needed. |
-1 |
-1 |
-1 |
Your PR is unmergeable. Please have a look and possibly rebase it. |
-1 |
-1 |
Fix ranges in the x-axis and errors for NumberOfClustersinFED_v_FEDID and NumberOfDigisinFED_v_FEDID