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

Switch GEANT4 IB to Geant4 10.6ref10 #49

Closed
civanch opened this issue Oct 29, 2020 · 12 comments
Closed

Switch GEANT4 IB to Geant4 10.6ref10 #49

civanch opened this issue Oct 29, 2020 · 12 comments

Comments

@civanch
Copy link
Contributor

civanch commented Oct 29, 2020

A new Geant4 reference version is available : ~vnivanch/public/geant4-10-06-ref-10
This Geant4 version includes main development for 2020.
We need to use this new version in GEANT4 special IB for early testing.

This time we would ask to build GEANT4 IB without usage of VecGeom.

@cmsbuild
Copy link

A new Issue was created by @civanch Vladimir Ivantchenko.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@civanch
Copy link
Contributor Author

civanch commented Nov 2, 2020

@mrodozov , @smuzaffar , we need to test new Geant4 version in order to be able promptly feedback Geant4. In parallel, this version if built without VecGeom will help testing DD4Hep - it is even more urgent.

@mrodozov
Copy link
Contributor

mrodozov commented Nov 2, 2020

is this change to be put on
https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_11_2_X/g4_106/geant4.spec
or on the branch that is currently using geant4 10.6.9 ?
https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_11_2_X/geant4/geant4.spec

@civanch
Copy link
Contributor Author

civanch commented Nov 2, 2020

@mrodozov , on the last one substituting 10.6.9. What is essential for DD4Hep validation is to disable VecGeom.

@smuzaffar
Copy link
Contributor

@mrodozov , this shoudl go in to https://github.com/cms-sw/cmsdist/blob/IB/CMSSW_11_2_X/geant4/geant4.spec branch (where we currently have geant4 10.6.9)

@mrodozov
Copy link
Contributor

mrodozov commented Nov 2, 2020

Yes, this is clear now.
@civanch , since when the cmake flag
-DGEANT4_USE_USOLIDS="all"
was expecting VecGeom should be available, I'd assume it shouldn't be used.
When I used it it searched for vecgeom and the config fails.
Instead I used
-DGEANT4_USE_USOLIDS-ADVANCED:INTERNAL=1
and it builds , but I can't find example cmake configs on github or gitlab .
Do you know more on that, is this a right way to be build ?
I remember something for Usolids being a substitute for VecGeom, so one or the other should always be used,
but some of the Usolids options are expecting VecGeom. There should be a way to use Usolids alone, correct ?

@mrodozov
Copy link
Contributor

mrodozov commented Nov 4, 2020

@civanch we have now this changes integrated in the last IB,
https://cmssdt.cern.ch/SDT/html/cmssdt-ib/#/relVal/CMSSW_11_2/2020-11-03-2300?selectedArchs=slc7_amd64_gcc820&selectedFlavors=GEANT4_X&selectedStatus=failed
14 relvals failing (looks like due to the changes)

@civanch
Copy link
Contributor Author

civanch commented Nov 4, 2020

@mrodozov , I am in contact with geometry experts of Geant4. This problem is known in Run-3 geometry, which we already fixed but not yet propagated to GT. But this should not appears in Run1 and Run2 WFs.

@civanch
Copy link
Contributor Author

civanch commented Nov 4, 2020

@mrodozov , I am running this new IB and it is working fine to me no problems, no crash. I do not yet understand if VecGeom is used or not used. It is a contradiction to what happens in standard IB WFs. What is confirmed: 10.6ref10 was built and runs indicate, that it is used.

@mrodozov
Copy link
Contributor

mrodozov commented Nov 4, 2020

@civanch the geant4 is built without vecgeom but I see I had to remove the vecgeom references from the toolfiles for scram

@mrodozov
Copy link
Contributor

mrodozov commented Nov 8, 2020

@civanch after cms-sw/cmsdist#6370 the last IB doesn't include vecgeom references anyhow.

@civanch
Copy link
Contributor Author

civanch commented Nov 8, 2020

@mrodozov , thanks!

@civanch civanch closed this as completed Nov 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants