-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Add an option to edmFileUtil to print information on clusters #48044
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
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-48044/44761 |
A new Pull Request was created by @makortel for master. It involves the following packages:
@Dr15Jones, @cmsbuild, @makortel, @smuzaffar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
@dan131riley @Dr15Jones If we want to use the information on the cluster boundaries or number of entries per cluster (or total size of all baskets in a cluster), this code shows how to do that. |
+1 Size: This PR adds an extra 24KB to repository Comparison SummarySummary:
|
7b336e8
to
5c11904
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-48044/44807 |
Pull request #48044 was updated. @Dr15Jones, @cmsbuild, @makortel, @smuzaffar can you please check and sign again. |
@cmsbuild, please test |
Comparison differences are related to #47071 |
5c11904
to
b2e64e2
Compare
Improved one comment |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-48044/44815 |
Pull request #48044 was updated. @Dr15Jones, @cmsbuild, @makortel, @smuzaffar can you please check and sign again. |
@cmsbuild, please test |
+1 Size: This PR adds an extra 24KB to repository Comparison SummarySummary:
|
Comparison differences are related to #47071 |
+core |
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. @sextonkennedy, @mandrenguyen, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR adds
--printClusters
command line option toedmFileUtil
. The information on the cluster boundaries, number of entries per cluster, and the total size of baskets in a cluster was useful in the investigation of #47750.Resolves cms-sw/framework-team#1386
PR validation:
Cluster information is getting printed on a random AOD file.