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

Use Message logger instead of std::cout. Remove spurious includes. Remov... #563

Merged
merged 1 commit into from Aug 19, 2013

Conversation

ianna
Copy link
Contributor

@ianna ianna commented Aug 19, 2013

  • Use Message logger instead of std::cout
  • Remove spurious includes
  • Remove commented out lines
  • Use auto conditions rather then specific GT

@ghost ghost assigned nclopezo Aug 19, 2013
@ktf
Copy link
Contributor

ktf commented Aug 19, 2013

@nclopezo please test.

@nclopezo
Copy link
Contributor

Hi,

I took these changes on top of CMSSW_7_0_X_2013-08-19-0200, I ran the unit tests and RelVals, all tests passed.

You can see the logs here:

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

ktf added a commit that referenced this pull request Aug 19, 2013
Use Message logger instead of std::cout. Remove spurious includes. Remov...
@ktf ktf merged commit e7f1e7d into cms-sw:CMSSW_7_0_X Aug 19, 2013
@ianna ianna deleted the replace-cout-with-message-logger branch October 26, 2015 16:33
veelken pushed a commit to veelken/cmssw that referenced this pull request Dec 21, 2016
cmsbuild pushed a commit that referenced this pull request Nov 10, 2020
Clean up ecalRawDecodingAndMultifit.py .
Rename ecalMultiFitUncalibRecHitGPUcfi.py to ecalMultiFitUncalibRecHitGPU_cfi.py .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants