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
[10_2_X] Update data-RecoBTag-Combined.spec #4550
[10_2_X] Update data-RecoBTag-Combined.spec #4550
Conversation
Backport of #4540
A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_10_2_X/gcc700. @cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks.
|
@cmsbuild please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
this confirms that this external can be merged by itself before the cms-sw/cmssw#25370 is merged. |
The tests are being triggered in jenkins. |
Comparison job queued. |
The tests are being triggered in jenkins. |
Comparison job queued. |
+externals this resolves #4537 |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_10_2_X/gcc700 IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
The tests are being triggered in jenkins. |
Comparison job queued. |
+externals |
@andrzejnovak @slava77 @perrotta this external update is needed by cms-sw/cmssw#25370 , that is not yet ready to be merged. As 10_2_9 should start today, is this integration causing issues if not matched to its CMSSW counterpart? I see in https://github.com/cms-data/RecoBTag-Combined/pull/19/files that some files are removed and a few slightly modified, but this might change the behaviour, am I correct? |
@fabiocos |
@slava77 sorry to be pedantic, is this test supposed to catch the changes induced by these data files? You answer would imply it is, they should appear in miniAOD output, am I correct? In such a case likely workflow 136.85 in the matrix should catch that? Where are these changes supposed to show an effect otherwise? |
ok, you comment "jenkins tests pass and comparisons with the baseline show changes only in slimmedJetsAK8 jet tags in an expected way: new tags are added and the existing ones are unchanged"... |
Sorry, I'm lost.
which I read as "can we integrate this data PR without cms-sw/cmssw#25370?" |
Backport of #4540