-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
2024 Data RelVals and InputInfo
Events Skimming (actually) Working
#45055
Conversation
cms-bot internal usage |
test parameters:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45055/40365
|
A new Pull Request was created by @AdrianoDee for master. It involves the following packages:
@sunilUIET, @AdrianoDee, @subirsarkar, @srimanob, @miquork, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: RelVals RelVals-INPUT RelValsThe relvals timed out after 4 hours.
Expand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ...
|
test parameters:
|
please test |
-1 Failed Tests: RelVals RelVals-INPUT
RelValsThe relvals timed out after 4 hours.
RelVals-INPUT
Expand to see more relval errors ...
|
Ok, some files are missing a copy to |
please test |
+1 Size: This PR adds an extra 12KB to repository Comparison SummarySummary:
|
+pdmv |
+Upgrade |
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. @antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
@AdrianoDee , looks like after this update many relvals are failing for 14.1.X Ibs.
|
@smuzaffar let me have a look |
PR description:
This PR proposes few things:
the introduction of a new script
das-up-to-nevents.py
that, given in input a dataset name, allows to get the list of files corresponding to N events. The dataset is also automatically skimmed to include only the run and the lumis from the latest and greatest Golden data certification json;a fix to the fact that the
InputInfo
class, while accepting as input a variableevents
does not really use it to skim the result of the das query. It's only really used in a log. I find it ambiguous and a bit confusing. In practice it also means that for 2023 and 2022 RelVals we have been running on O(few millions) events per pre-release (see e.g. CMSSW_14_1_0_pre3__Data_Run3_2023_Data-RunEGamma2023D). While this is nice in terms of being able to do that (kudos on computing!) it usually slows down a lot the RelVal production. This leverages on the script introduced in (1).the addition of 2024 data RelVals for Eras from B to F as
standard
wfs with2024.*
prefix. These are run with:Run3_2024
era;HLT:@relval2024
(currently pointing at GRun);a new set of RelVals under
relvals_data_highstats.py
that should include all the higher statistics data RelVals workflows with 150k,250k,500k and 1M events. For the moment it includes only the 2024 wfs. This set is not turned on by default to avoid to have an over crowded matrix.All the needed input RAW samples have been copied via Rucio at
T2_CH_CERN
.PR validation:
Running (e.g.)
runTheMatrix.py -w data,standard -l 12024.003015, 2024.00405, 2024.007005, 2024.0091, 2024.012005, 2024.013025, 2024.10305, 2024.108005, 2024.108015, 2024.210025, 2024.211015, 2024.2111, 2024.212025, 2024.213015, 2024.214025, 2024.21405, 2024.303015, 2024.310025, 2024.31305, 2024.3141, 2024.400005, 2024.402015