Skip to content
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

Migrate master from Geant4 11.1.2 to 11.2.1 #9085

Closed
civanch opened this issue Mar 21, 2024 · 6 comments · Fixed by #9086
Closed

Migrate master from Geant4 11.1.2 to 11.2.1 #9085

civanch opened this issue Mar 21, 2024 · 6 comments · Fixed by #9086

Comments

@civanch
Copy link
Contributor

civanch commented Mar 21, 2024

The new Geant4 11.2.1 is validated by SIM group (https://indico.cern.ch/event/1394263/ ). It is shown that there are no technical problems and no difference in physics results. CPU/memory analysis shows minor advantage (CPU - 0-3% depending on WF, less memory - see https://nhoerman.web.cern.ch/nhoerman/cms_simulation/olsky-05/Summary_MinBias_TTbar_SMS-T1tttt_ZEE_SIM_CMSSW_14_1_G4VECGEOM_8_thread_run-3_FullSim.html ).

We propose switch Geant4 master to variant tested in G4VECGEOM branch, which includes Geant4 itself and datasets G4EMLOW-8.5, G4ABLA-3.3, G4INCL-1.2.

We expect regular validation within 14_1_0_pre2 without creation of a parallel IB and without extra validation campaign.

@cmsbuild
Copy link
Contributor

cms-bot internal usage

@cmsbuild
Copy link
Contributor

A new Issue was created by @civanch.

@smuzaffar, @rappoccio, @antoniovilela, @sextonkennedy, @Dr15Jones, @makortel can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@smuzaffar
Copy link
Contributor

The new Geant4 11.2.1 is validated by SIM group (https://indico.cern.ch/event/1394263/ )

@civanch , this sim validation was done based on 14.0.G4Vecgeom IBs ... right?

We expect regular validation within 14_1_0_pre2 without creation of a parallel IB and without extra validation campaign.

Are you suggesting to update Geant4 11.2.1 directly in 14.1.X master IB without the need of CMSSW_14_1_0_pre2_G4VECGEOM release and its validation?

@civanch
Copy link
Contributor Author

civanch commented Mar 22, 2024

@smuzaffar , we make validation on top of 14_1_X_G4VECGEOM branch.

Because we do not see a change on level of our tests, we propose to include 11.2.1 into 14_1_X master and validate with 14_1_0_pre2 without creation extra IB. We do not expect full regressing but this release of Geant4 is really technical.

@smuzaffar
Copy link
Contributor

we propose to include 11.2.1 into 14_1_X master and validate with 14_1_0_pre2 without creation extra IB.

@civanch , "without creation extra IB" you mean extrra/special CMSSW_14_1_0_pre2_G4VECGEOM release. .... right?
So if I understand correctly, we want to update Geant4 in 14.1.X master from current geant4 version 11.1.2 to 11.2.1 and just build 14.1.0.pre2 with it and validate it .... right?

@civanch
Copy link
Contributor Author

civanch commented Mar 22, 2024

@smuzaffar , yes, exactly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants