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
Issue fix in Exotica HLT validation code #34006
Conversation
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34006/23128
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34006/23129
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34006/23130
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34006/23131
|
A new Pull Request was created by @borzari for master. It involves the following packages: HLTriggerOffline/Exotica @andrius-k, @kmaeshima, @ErnestaP, @ahmad3213, @cmsbuild, @jfernan2, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-f6aa4a/15743/summary.html Comparison SummarySummary:
|
+1 |
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. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
This is an issue fix of the Exotica HLT validation code. The mentioned issue is #32550.
To summarize: there was an index running from 0 to the
matches
vector sizematches.size()
, that was fetching for elements inside the vectordxys
, while there was no guaranteed one to one correspondence between their sizes incmssw/HLTriggerOffline/Exotica/src/HLTExoticaPlotter.cc
Lines 151 to 153 in 393431e
Some dedicated counters to correctly fill
dxys
and get its elements were created.The code was tested using local files and showed no more discrepancy between the size of the
dxys
vector with the index that get its elements. The commandrunTheMatrix.py -l limited -i all --ibeos
was run with success.