Adding P2 L1T NNVtx#10605
Conversation
|
A new Pull Request was created by @BenjaminRS for branch IB/CMSSW_17_0_X/master. @akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
please test with cms-sw/cmssw#51008 |
|
-1 Failed Tests: RelVals Failed RelVals |
|
@BenjaminRS , you also need to add scram tool file (cmsdist/scram-tools.file/tools/NNVtx/nnvtx.xml) for this new external ( see https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_20_0_X/master/scram-tools.file/tools/CICADA/cicada.xml or https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_20_0_X/master/scram-tools.file/tools/AXOL1TL/axol1tl.xml as exmaples). Also you need to add this new external as dependency of cmssw in https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_20_0_X/master/cmssw-tools.spec |
|
Pull request #10605 was updated. |
|
please test with cms-sw/cmssw#51008 |
|
Hi @smuzaffar -- thank you for the tips. I set the |
|
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ebd0ad/53761/summary.html Failed External BuildI found compilation warning when building: See details on the summary page. |
|
@BenjaminRS , ideally this should have the By the way, looks like https://github.com/cms-hls4ml/NNVtx does not contin tag v1.0.0.tar.gz. build fails to download the sources |
|
-1 Failed Tests: RelVals Failed RelVals |
|
please test with cms-sw/cmssw#51008 20.1.X IB is available now |
|
-1 Failed Tests: RelVals The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Failed RelVals |
|
please test with cms-sw/cmssw#51008
|
|
-1 Failed Tests: RelVals Failed RelVals |
|
@BenjaminRS , I only see |
|
@BenjaminRS , I see that you have fixed the build system cms-hls4ml/NNVtx#2 but we need a new tag of NNVtx with your fix and update NNVtx.spec here to use the new tag |
|
Hi @smuzaffar - this is what this morning's PR fixed: cms-hls4ml/NNVtx#2 |
|
Pull request #10605 was updated. |
|
please test with cms-sw/cmssw#51008
|
|
-1 Failed Tests: RelVals The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Failed RelVals |
Adding into cmsdist the spec file for the Phase-2 Level-1 Trigger NNVtx Vertex Reconstruction.
The hls4ml files are kept here: https://github.com/cms-hls4ml/NNVtx
The gitlab page is here: https://gitlab.cern.ch/ml_l1/l1-vertex-finding