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

DQM SiStrip endcap naming #567

Merged

Conversation

mtosi
Copy link
Contributor

@mtosi mtosi commented Aug 19, 2013

  • update config files used by QualityTest
  • adjust c++11 loop in EventFilter/SiStripRawToDigi/test/plugins/SiStripClustersDSVBuilder.cc

NB: w/ this patch some SiStrip MEs in the DQM root file change their name (and directory)
=> this root file is not compatible w/ the prevoius ones

@thspeer
Copy link
Contributor

thspeer commented Aug 20, 2013

Working @thspeer

@ghost ghost assigned deguio Aug 20, 2013
@cmsbuild
Copy link
Contributor

The following categories have been signed by speer (a.k.a. @thspeer on GitHub): Reconstruction

@cms-git-reconstruction

@cmsbuild
Copy link
Contributor

The following categories have been signed by @deguio: DQM

@cms-git-dqm

@ktf
Copy link
Contributor

ktf commented Aug 22, 2013

@nclopezo can you please test this?

@nclopezo
Copy link
Contributor

Hi,

I tested it on top of CMSSW_7_0_X_2013-08-22-0200, all tests passed:

you can see the logs here:

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/210/console

ktf added a commit that referenced this pull request Aug 22, 2013
@ktf ktf merged commit f4c68ce into cms-sw:CMSSW_7_0_X Aug 22, 2013
nclopezo pushed a commit to nclopezo/cmssw that referenced this pull request Apr 30, 2014
fwyzard pushed a commit to fwyzard/cmssw that referenced this pull request Nov 16, 2020
Factor out the chi2 computation from the ECAL multifit and HCAL MAHI code,
and move it to MultifitComputations.
fwyzard pushed a commit to fwyzard/cmssw that referenced this pull request Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants