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
Add GEM recHits and segments to AlCaRecos #38859
Add GEM recHits and segments to AlCaRecos #38859
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38859/31269
|
A new Pull Request was created by @hyunyong for master. It involves the following packages:
@malbouis, @yuanchao, @cmsbuild, @francescobrivio, @ChrisMisan, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
@hyunyong you dont have the numbers on the size increase, do you? |
@cmsbuild , please test |
Please also modify the PR title |
No, I don't know but I will ask it at the GEM DPG meeting tomorrow. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-654334/26453/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-38859/31281
|
Pull request #38859 was updated. @malbouis, @yuanchao, @cmsbuild, @francescobrivio, @ChrisMisan, @tvami can you please check and sign again. |
@cmsbuild , please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-654334/26471/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
I have checked the output and the MuAlCalIsolated skim contains GEM recHits and segments now. |
That's great, can we please have a the numbers for the increase of size before merging the PR? |
Yes, I processed 1,000 events and the size difference is 77894 bytes. So, average size of GEM recHits and segments is 78 bytes/event. |
So I believe that's a 2% increase, for an ALCARECO, that's probably fine. Ok, I'll sign it now, and please prepare the backport |
+alca
|
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. @perrotta, @dpiparo, @qliphy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
Yes, I made the backport #38876 |
+1 |
PR description:
Add GEM recHits and segments in ALCARECOMuAlCalIsolatedMu, ALCARECOMuAlGlobalCosmics, and ALCARECOMuAlGlobalCosmicsInCollisions.
Need to backport to 12_4_X.