[Mini from mini] v2-v4 to v6 full recipe#49098
Conversation
…o use original packedPFCandidates collection in miniAODFromMiniAOD_customizeCommon()
…ith cms-merge-topic
|
cms-bot internal usage |
|
A new Pull Request was created by @vlimant for master. It involves the following packages:
@cmsbuild, @ctarricone, @davidlange6, @fabiocos, @ftenchini, @ftorrresd, @gabrielmscampos, @hqucms, @jfernan2, @mandrenguyen, @nothingface0, @rseidita can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
enable nano |
|
please test |
|
please test if I understood correctly, the cache the is updated and the bot should get files properly |
bot cache should be up to date for workflow 2500.0204 rest might still fail |
|
is this still failing on the AODSIM files ? should the test be aborted and started over ? |
|
-1 Failed Tests: RelVals-INPUT RelVals-NANO RelVals-INPUT
Expand to see more relval errors ...RelVals-NANO
Expand to see more relval errors ...Comparison SummarySummary:
|
|
something not functioning here. @smuzaffar any clue please ? |
|
@vlimant , there root files which tests try to access are still only available on tape [a]. Only workflow [a] [b] |
|
Just sharing a comparison for the same 1k events (#49137 is included).
|
|
yes, indeed, in the storm of things I dreamt this passed locally. There's a --data missing that makes the difference. |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49098/46458
|
|
Pull request #49098 was updated. @AdrianoDee, @DickyChant, @Moanwar, @antoniovagnerini, @battibass, @cmsbuild, @davidlange6, @fabiocos, @ftenchini, @ftorrresd, @mandrenguyen, @miquork, @srimanob, @subirsarkar can you please check and sign again. |
|
please test beats me for the missing inputs, the files are accessible |
|
@smuzaffar regarding #49098 (comment) @AdrianoDee mentioned #49098 (comment) ; the ib query lists files from anywhere, instead of T2_CH_CERN das query results in but the actual list is in https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-43dfc2/48643/runTheMatrixNANO-results/2500.0202_NANOmcUL16reMINI/step1_dasquery.log is very different hence the repeated issue. |
|
-1 Failed Tests: RelVals-INPUT RelVals-NANO RelVals-INPUT
RelVals-NANO
Comparison SummarySummary:
|
|
again the failure is unrelated the changes proposed here. |
|
@vlimant , bot das cache for 3 failing relvals is now updated. As I wrote earlier bot only force update its cache after 7 days if it already has a valid result (non-empty) . Anyway, I have forced run das-query jenkins job to update cache for these 3 relvals. I have now restarted failing jobs to use the newer cache, hopefully things should work this time |
|
thanks @smuzaffar |
@smuzaffar, thanks! Since we are here, something I don't understand yet is how the original DAS cached result was non-empty in the first place, given the (I tested an analogous das query locally, and it would give no result) |
|
@AdrianoDee , In past (many many years ago) we wanted IB's relvals to access only data at CERN, so we had added das-query cache script works on the above assumption. So if it can not find any data with So for issue we have seen here was that though das was returning a list of files but they were not available on disk or not accessible via AAA. So cmsRun was not able to read those files and bot was not able to cache the root files. |
|
by the way, all 3 failing workflows now passed |
|
Clear, thanks a lot. |
|
+1 Size: This PR adds an extra 24KB to repository Comparison SummarySummary:
Nano size comparison Summary:
NANO Comparison SummaryThe workflows 2500.0204, 2500.0203, 2500.0202, 2500.0201, 2500.0003, 2500.0002 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons Summary:
|
|
@cms-sw/xpog-l2 @cms-sw/pdmv-l2 : please go ahead and review/sign so that we get it in the 16.0 dev cycle, and add work on top to fix the nano vs nano discrepancies |
|
+Upgrade |
|
+pdmv |
|
+xpog |
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
11 participants
PR description:
this PR builds on top of #48988 with the addition of the cmsDriver integration
the mini-from-mini is configured with:
cmsDriver.py step2 -s PAT:Configuration/StandardSequences/REMINI_cff.patAlgosToolsTask --era Run3 -n 100 --process REMINI --conditions auto:phase1_2024_realistic --mc --scenario pp --eventcontent MINIAODSIM --datatier MINIAODSIM --filein /store/mc/Run3Summer23MiniAODv4/TTtoLNu2Q_TuneCP5_13p6TeV_powheg-pythia8/MINIAODSIM/130X_mcRun3_2023_realistic_v14-v2/2520000/373b1353-c6b4-4ec0-bc13-b5c0eff8c189.root --fileout file:step2.rootsuggestions to a different syntax are welcome, and I'll update the PR
PR validation:
the limited matrix ran to completion, with some unrelated errors related to file access.
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
Backports to 15.1 and 15.0 are expected