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

change the naming convention of CaloMETs #3942

Merged
merged 29 commits into from May 22, 2014

Conversation

TaiSakuma
Copy link
Contributor

As the PR #3801 was closed with unmerged commits, this PR requests similar commits for 72X

This PR is mergeable at the moment. However, since this PR involves changes in many packages, it will be probably soon unmergeable. I would like to request to merge this PR soon while it is still mergeable.

@cmsbuild
Copy link
Contributor

@slava77
Copy link
Contributor

slava77 commented May 21, 2014

Comparison with the baseline    Still running...

they probably need a nudge to finish

@cmsbuild
Copy link
Contributor

@slava77
Copy link
Contributor

slava77 commented May 22, 2014

+1

for #3942 8af6a81
jenkins OK
also checked in CMSSW_7_2_X_2014-05-21-1400 that the code is essentially the same as #3801 (the same for RECO purposes).

ktf added a commit that referenced this pull request May 22, 2014
Reco -- Change the naming convention of CaloMETs
@ktf ktf merged commit a2aac79 into cms-sw:CMSSW_7_2_X May 22, 2014
@ktf
Copy link
Contributor

ktf commented May 22, 2014

Bypassing DQM, complain if not ok.

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

5 participants