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

Adding AK4PF JEC to Run2 MC and data GTs. #4555

Merged
merged 1 commit into from
Jul 9, 2014

Conversation

diguida
Copy link
Contributor

@diguida diguida commented Jul 8, 2014

New set of GTs with the following additions:

  • Run2 design GT:
    • adding AK4PF and AK4PFchs JEC
  • Run2 MC GTs (both 50 and 25 ns):
    • adding AK4PF and AK4PFchs JEC
  • Run1 data GT:
    • adding AK4PF and AK4PFchs JEC
  • Run2 data GT:
    • adding AK4PF and AK4PFchs JEC

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 8, 2014

A new Pull Request was created by @diguida (Salvatore Di Guida) for CMSSW_7_2_X.

Adding AK4PF JEC to Run2 MC and data GTs.

It involves the following packages:

Configuration/AlCa

@cmsbuild, @Degano, @diguida, @rcastello, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

@ktf
Copy link
Contributor

ktf commented Jul 8, 2014

Approving since this is currently broken.

@ktf
Copy link
Contributor

ktf commented Jul 8, 2014

Nevermind. Let's first see tests.

@diguida
Copy link
Contributor Author

diguida commented Jul 8, 2014

@ktf I'd like to see tests first. In the meantime, with @apfeiffer1 we are debugging using valgrind

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 8, 2014

@diguida
Copy link
Contributor Author

diguida commented Jul 8, 2014

@ktf The short matrix did not run any of the failing wf in runTheMatrix. I have observed failures in wf 4.71, 25400.0 and 25401.0.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 8, 2014

@davidlange6
Copy link
Contributor

still issues on gcc490 with many workflows - but do not believe it has anything to do with this PR - so merging now to at least remove the ak4 errors

davidlange6 added a commit that referenced this pull request Jul 9, 2014
Adding AK4PF JEC to Run2 MC and data GTs.
@davidlange6 davidlange6 merged commit 22cef80 into cms-sw:CMSSW_7_2_X Jul 9, 2014
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.

4 participants