Update Sherpa to v3 by shimashimarin · Pull Request #10572 · cms-sw/cmsdist

shimashimarin · 2026-05-21T11:28:47Z

Sherpa version 3 have been released last year Sherpa release. This is a big jump of Sherpa version from v2.2.16 to v3.0.4:

Switch to cmake
Add libzip for Sherpa3
Add openloops inlcude folder for compiling Sherpa3

cmsbuild · 2026-05-21T11:29:14Z

A new Pull Request was created by @shimashimarin for branch IB/CMSSW_17_0_X/master.

@akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

cmsbuild · 2026-05-21T11:29:14Z

cms-bot internal usage

smuzaffar · 2026-05-21T11:37:31Z

@@ -0,0 +1,16 @@
+### RPM external libzip 1.11.4


@shimashimarin , @akritkbehera already have added libzip (which was needed for xrootd 6.x series) in devel branch https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_17_0_X/devel/libzip.spec . Can you please use https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_17_0_X/devel/libzip.spec so that we do not have to resolve any merge conflicts

smuzaffar · 2026-05-21T11:47:26Z

@@ -0,0 +1,10 @@
+<tool revision="1">


@shimashimarin , is sherpa v3 linking against libzip.so? If yes then please add the following here

<lib name="zip"/>

smuzaffar · 2026-05-21T11:49:17Z

+        <environment name="INCLUDE" default="$TOOL_BASE/include"/>
+    </client>
+    <runtime name="PATH" value="$TOOL_BASE/bin" type="path"/>
+    <runtime name="ROOT_INCLUDE_PATH" value="$INCLUDE" type="path"/>


ROOT_INCLUDE_PATH should only be added if the headers files of this tool end up in root dictionaries? Do you think that libzip headers will will end up in root dicts generated in cmssw (directly or indirectly via sherpa headers)? If not then please remove this line

smuzaffar · 2026-05-21T11:54:59Z


 %prep
 %setup -n bzip2-bzip2-%{realversion}
+sed -i 's/-Wl,-soname -Wl,libbz2.so.1.0/-Wl,-soname -Wl,libbz2.so.1/' Makefile-libbz2_so


@shimashimarin , I think this is not needed if you use the libzip spec from https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_17_0_X/devel/libzip.spec . @akritkbehera mentioned that if few flags are not explicitly turned OFF for libzip then it picks up system bz2lib and that is why rpm dependnecy fails. Please use the libzip from devel branch and remove this line

@smuzaffar thanks for you comments. I've update libzip.spec and modified related things accordingly

cmsbuild · 2026-05-21T14:01:04Z

Pull request #10572 was updated.

smuzaffar · 2026-05-21T14:43:46Z

please test

cmsbuild · 2026-05-21T16:35:41Z

-1

Failed Tests: Build
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b9c065/53413/summary.html
COMMIT: 9fac979
CMSSW: CMSSW_17_0_X_2026-05-21-1100/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/10572/53413/install.sh to create a dev area with all the needed externals and cmssw changes.

Failed Build

I found compilation error when building:

                 from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc13/external/sherpa/3.0.4-27726c334c06f1346e0aa71297dd8d93/include/SHERPA-MC/ATOOLS/Phys/Blob_List.H:4,
                 from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc13/external/sherpa/3.0.4-27726c334c06f1346e0aa71297dd8d93/include/SHERPA-MC/SHERPA/Main/Filter.H:6,
                 from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc13/external/sherpa/3.0.4-27726c334c06f1346e0aa71297dd8d93/include/SHERPA-MC/SHERPA/Main/Sherpa.H:7,
                 from src/GeneratorInterface/SherpaInterface/src/SherpaHepMC3Hadronizer.cc:9:
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc13/external/sherpa/3.0.4-27726c334c06f1346e0aa71297dd8d93/include/SHERPA-MC/ATOOLS/Phys/Flavour.H: In constructor 'ATOOLS::Particle_Info::Particle_Info()':
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc13/external/sherpa/3.0.4-27726c334c06f1346e0aa71297dd8d93/include/SHERPA-MC/ATOOLS/Phys/Flavour.H:27:30: error: 'ATOOLS::Particle_Info::m_radius' will be initialized after [-Werror=reorder]
   27 |     double  m_mass, m_hmass, m_radius, m_yuk, m_width, m_dg, m_dm, m_qoverp2;
      |                              ^~~~~~~~
/data/cmsbld/jenkins/workspace/ib-run-pr-tests/testBuildDir/el8_amd64_gcc13/external/sherpa/3.0.4-27726c334c06f1346e0aa71297dd8d93/include/SHERPA-MC/ATOOLS/Phys/Flavour.H:27:21: error:   'double ATOOLS::Particle_Info::m_hmass' [-Werror=reorder]
   27 |     double  m_mass, m_hmass, m_radius, m_yuk, m_width, m_dg, m_dm, m_qoverp2;
      |                     ^~~~~~~

smuzaffar · 2026-05-21T16:39:31Z

@shimashimarin , looks like we need changes in https://github.com/cms-sw/cmssw for new sherpa

shimashimarin · 2026-05-21T17:01:48Z

yes, the SherpaInterface should be revised. Sherpa v3 is very different from Sherpa v2. I will update SherpaInterface meanwhile

shimashimarin · 2026-05-22T08:15:08Z

@smuzaffar forget mentioning that, I had another PR for updating SherpaInterface of Sherpa2: cms-sw/cmssw#50955. This is very useful for current Sherpa sample production. If it's merged, I will try to work on Sherpa3 CMSSW interface

cmsbuild · 2026-06-02T22:18:06Z

Pull request #10572 was updated.

cmsbuild · 2026-06-03T12:28:24Z

Pull request #10572 was updated.

CUDA Toolkit 12.9 Update 2 includes critical cuBLAS fixes for issues that could lead to incorrect results, hangs, or invalid memory access in specific cuBLASLtMatmul() configurations (notably ALGO_CONFIG_ID 66 across multiple compute capabilities), as well as a ztrmm kernel correctness issue on NVIDIA Ada and Blackwell GeForce-class GPUs. See https://docs.nvidia.com/cuda/archive/12.9.2/cuda-toolkit-release-notes/index.html for the release notes, bug fixes and known issues.

…owPtUtilities-to-V00-01-00 Update tag for RecoTracker-PixelLowPtUtilities to V00-01-00

…UDA_12_9_2 [17.0] Update CUDA to version 12.9.2

…es-V00-01-00 [17.0.X] Update tag for RecoTracker-PixelLowPtUtilities to V00-01-00

This overrides the value of ROCM_PATH that may be set in the system environment and point to e.g. /opt/rocm, which can break tools like amd-smi.

…OCM_PATH [17.0] Set `ROCM_PATH` to the CMSSW ROCm installation

…0_X/master Update L1TSC4NGJet model to v2.0.0

Wires the new cms-hls4ml/NNPuppiTauModel package into the SCRAM build following the L1TSC4NGJetModel/AXOL1TL precedent, so TauNNIdHW can load its weights via hls4mlEmulator::ModelLoader instead of in-tree global arrays (avoids the symbol-clobbering bug class in cms-sw/cmssw#49632). Co-authored-by: artlbv <arturlbv@gmail.com>

v1.0.0 was missing NNPuppiTauModel_v1/NN/weights/w2.h, which broke the external build (parameters.h includes it). v1.0.1 fixes this. Co-authored-by: artlbv <arturlbv@gmail.com>

Co-authored-by: Malik Shahzad Muzaffar <shahzad.malik.muzaffar@cern.ch>

Get the sources from git to support submodules.

backport of cms-sw#10690

[17.0]Update tag for L1Trigger-L1TMuon to V01-13-00

…CX_v1.21.0 [17.0] Update UCX to version 1.21.0

[17.0] Python packages update to get security fixes

[17.0.X] Update DQM-Integration version to V00-18-02

…ation_CMSSW_17_0_X [Backport] Add NNPuppiTauModel external spec and toolfile

cmsbuild · 2026-07-30T18:33:52Z

Pull request #10572 was updated.

shimashimarin added 2 commits May 20, 2026 23:35

sherpa3

7143712

fix sherpa denpendce

a940c2d

cmsbuild added tests-pending externals-pending pending-signatures orp-pending labels May 21, 2026

smuzaffar reviewed May 21, 2026

View reviewed changes

minor adjusts

9fac979

cmsbuild added tests-started and removed tests-pending labels May 21, 2026

cmsbuild added tests-rejected and removed tests-started labels May 21, 2026

Merge branch 'cms-sw:IB/CMSSW_17_0_X/master' into IB/CMSSW_17_0_X/master

4e006c4

cmsbuild added tests-pending and removed tests-rejected labels Jun 2, 2026

1st try add both sherpa2 & sherpa3

81f96b8

smuzaffar changed the base branch from IB/CMSSW_17_0_X/master to IB/CMSSW_20_0_X/master June 5, 2026 16:50

akritkbehera changed the base branch from IB/CMSSW_20_0_X/master to IB/CMSSW_20_1_X/master June 9, 2026 07:53

cmsbuild and others added 25 commits June 10, 2026 11:36

Merge pull request cms-sw#10618 from cms-sw/update-RecoTracker-PixelL…

d042bb9

…owPtUtilities-to-V00-01-00 Update tag for RecoTracker-PixelLowPtUtilities to V00-01-00

Merge pull request cms-sw#10610 from fwyzard/IB/CMSSW_17_0_X/master_C…

fb7491e

…UDA_12_9_2 [17.0] Update CUDA to version 12.9.2

Merge pull request cms-sw#10621 from mroguljic/170X_PixelLowPtUtiliti…

d7c5d6f

…es-V00-01-00 [17.0.X] Update tag for RecoTracker-PixelLowPtUtilities to V00-01-00

Set ROCM_PATH to the CMSSW ROCm installation

10ec62c

This overrides the value of ROCM_PATH that may be set in the system environment and point to e.g. /opt/rocm, which can break tools like amd-smi.

Merge pull request cms-sw#10651 from fwyzard/IB/CMSSW_17_0_X/master_R…

0edb2e4

…OCM_PATH [17.0] Set `ROCM_PATH` to the CMSSW ROCm installation

Update L1TSC4NGJet model to v2.0.0

0fa1981

Merge pull request cms-sw#10667 from CMS-L1T-Jet-Tagging/IB/CMSSW_17_…

a866189

…0_X/master Update L1TSC4NGJet model to v2.0.0

Bump NNPuppiTauModel to v1.0.1

ee8ada2

v1.0.0 was missing NNPuppiTauModel_v1/NN/weights/w2.h, which broke the external build (parameters.h includes it). v1.0.1 fixes this. Co-authored-by: artlbv <arturlbv@gmail.com>

Update scram-tools.file/tools/NNPuppiTauModel/nnpuppitaumodel.xml

2efedb4

Co-authored-by: Malik Shahzad Muzaffar <shahzad.malik.muzaffar@cern.ch>

Add default package name definition for vectorization

8234219

Update cmsdist_packages.py

cecea65

added override_microarch_name for vectorized packages

03f1371

UCX version 1.20.1; enabled multi-vectorization

3f8909a

Update UCX to version 1.21.0

08db9e8

Get the sources from git to support submodules.

Python packages update to get security fixes

36d06d7

Added upyter-builder as dependency of jupyterlab

73d7f82

[17.0.X]Update tag for L1Trigger-L1TMuon to V01-13-00

da090d5

backport of cms-sw#10690

Merge pull request cms-sw#10694 from cms-sw/smuzaffar-patch-7

9be746a

[17.0]Update tag for L1Trigger-L1TMuon to V01-13-00

Merge pull request cms-sw#10689 from fwyzard/IB/CMSSW_17_0_X/master_U…

55238f6

…CX_v1.21.0 [17.0] Update UCX to version 1.21.0

Merge pull request cms-sw#10693 from cms-sw/backport-10674-170x

86ba07a

[17.0] Python packages update to get security fixes

[17.0.X] Update DQM-Integration version to V00-18-02

2a5236d

Merge pull request cms-sw#10702 from cms-sw/smuzaffar-patch-10

81de568

[17.0.X] Update DQM-Integration version to V00-18-02

Merge pull request cms-sw#10679 from artlbv/nnpuppitau-externals-migr…

922b560

…ation_CMSSW_17_0_X [Backport] Add NNPuppiTauModel external spec and toolfile

Merge branch 'cms-sw:IB/CMSSW_17_0_X/master' into IB/CMSSW_17_0_X/master

70b1cd9

Uh oh!

Conversation

shimashimarin commented May 21, 2026

Uh oh!

cmsbuild commented May 21, 2026

Uh oh!

cmsbuild commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

smuzaffar May 21, 2026

Choose a reason for hiding this comment

Uh oh!

smuzaffar May 21, 2026

Choose a reason for hiding this comment

Uh oh!

smuzaffar May 21, 2026

Choose a reason for hiding this comment

Uh oh!

smuzaffar May 21, 2026

Choose a reason for hiding this comment

Uh oh!

shimashimarin May 21, 2026

Choose a reason for hiding this comment

Uh oh!

cmsbuild commented May 21, 2026

Uh oh!

smuzaffar commented May 21, 2026

Uh oh!

cmsbuild commented May 21, 2026

Failed Build

Uh oh!

smuzaffar commented May 21, 2026

Uh oh!

shimashimarin commented May 21, 2026

Uh oh!

shimashimarin commented May 22, 2026

Uh oh!

cmsbuild commented Jun 2, 2026

Uh oh!

cmsbuild commented Jun 3, 2026

Uh oh!

cmsbuild commented Jul 30, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

cmsbuild commented May 21, 2026 •

edited

Loading