Rename SimDataformats TICL associator as TICLAssociationMap - #51649
Conversation
|
cms-bot internal usage |
|
A new Pull Request was created by @sbaldu for master. It involves the following packages:
@Moanwar, @battibass, @civanch, @cmsbuild, @ctarricone, @ftorrresd, @gabrielmscampos, @jfernan2, @kpedro88, @lviliani, @mandrenguyen, @mdhildreth, @mkirsano, @rseidita, @sensrcn, @srimanob, @theofil can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
please test |
|
-1 Failed Tests: UnitTests Failed Unit TestsI found 1 errors in the following unit tests: ---> test testMakeHGCalValidationPlots had ERRORS Comparison SummarySummary:
|
|
From the log https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b0a44d/55174/unitTests/log.txt Seems like a problem with the input files? |
|
please test
|
|
+generators |
|
+1 |
|
+xpog |
|
+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. @mandrenguyen, @sextonkennedy, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
|
@sbaldu , some unit tests start failing with error [a] . Looks like change in this PR is causing such issues in IBs. Can you please look in to it? |
Yes I will look into that |
FYI @Electricks94 |
@sbaldu FYI this is also discussed at #51756. |
PR description:
PR #49995 introduces a new heterogeneous Association Map under the
ticlnamespace. This PR renames theticl::AssociationMapdefined underSimDataFormats/Associationsto avoid clashes inside TICL code.FYI @waredjeb