use G4 10.7 version of LogicalBorderSurface on a G4 10.6 baseline

Ciao @gcorti this is the draft MR we have been discussing. It uses the G4 10.7 version of LogicalBorderSurface on the current master using the G4 10.6 baseline. My patch was off tag v10r6p2t6 but I realised that this is identical to the current master, so this is a MR against master. I only tested that it builds (in the ARM build) and that Gauss was then also building (but it needed a rebuild). Let's see if this works and if it gives any performance benefit... Thanks Andrea

changed title from Draft: use G4 10.7 version of LogigalBorderSurface on a G4 10.6 baseline to Draft: use G4 10.7 version of LogicalBorderSurface on a G4 10.6 baseline

added 1 deleted label

mentioned in issue #10 (closed)

See #10 (closed) for a detailed discussion of some first results. There seemed to be an interesting speedup, but the numbers are not reproducible, they are very unstable. No clear evidence of a speedup (or a slowdown) at this point in time.

• [2023-01-19 00:03] Validation started with lhcb-gauss-dev#3405
• [2023-01-20 00:03] Validation started with lhcb-gauss-dev#3406
• [2023-01-21 00:04] Validation started with lhcb-gauss-dev#3407
• [2023-01-21 10:36] Validation started with lhcb-gauss-dev#3408
• [2023-01-22 00:04] Validation started with lhcb-gauss-dev#3409
• [2023-01-23 00:03] Validation started with lhcb-gauss-dev#3410
• [2023-01-24 00:04] Validation started with lhcb-gauss-dev#3411
• [2023-01-25 00:04] Validation started with lhcb-gauss-dev#3412
• [2023-01-26 00:05] Validation started with lhcb-gauss-dev#3413
• [2023-01-27 00:04] Validation started with lhcb-gauss-dev#3414
• [2023-01-28 00:04] Validation started with lhcb-gauss-dev#3415
• [2023-01-29 00:04] Validation started with lhcb-gauss-dev#3416
• [2023-01-30 00:03] Validation started with lhcb-gauss-dev#3417
• [2023-01-31 00:05] Validation started with lhcb-gauss-dev#3418
• [2023-02-01 00:05] Automatic merge failed in [lhcb-g4-dev#1472](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1472/Geant4/checkout
• [2023-02-01 00:06] Validation started with lhcb-gauss-lamarr#712
• [2023-02-01 00:06] Validation started with lhcb-gauss-gen-dev#1189
• [2023-02-01 00:06] Validation started with lhcb-gauss-dev#3419
• [2023-02-01 00:06] Automatic merge failed in [lhcb-gaussino-fastsim#470](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/470/Geant4/checkout
• [2023-02-01 00:06] Validation started with lhcb-gaussino#1431
• [2023-02-01 00:06] Validation started with lhcb-gaussino-dev#13
• [2023-02-01 00:11] Validation started with lhcb-head-2#771
• [2023-02-01 00:12] Validation started with lhcb-gaudi-head#3462
• [2023-02-01 00:13] Validation started with lhcb-head#3482
• [2023-02-01 00:16] Validation started with lhcb-lcg-dev4lhcb#48
• [2023-02-01 00:17] Validation started with lhcb-lcg-dev3lhcb#48
• [2023-02-01 00:17] Validation started with lhcb-run3-cleanup#731
• [2023-02-01 00:18] Validation started with lhcb-sim10-gaudi-head#150
• [2023-02-01 00:20] Validation started with lhcb-sim10#1446
• [2023-02-01 07:28] Validation started with lhcb-gaussino-dev#14
• [2023-02-01 07:28] Automatic merge failed in [lhcb-gaussino-fastsim#471](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/471/Geant4/checkout
• [2023-02-01 07:32] Validation started with lhcb-gaussino#1432
• [2023-02-01 07:32] Validation started with lhcb-gaudi-head#3463
• [2023-02-01 07:36] Validation started with lhcb-head#3483
• [2023-02-01 07:55] Validation started with lhcb-run3-cleanup#732
• [2023-02-01 10:20] Validation started with lhcb-sim10#1447
• [2023-02-01 14:42] Validation started with lhcb-dd4hep#1390
• [2023-02-02 00:05] Validation started with lhcb-gauss-gen-dev#1190
• [2023-02-02 00:05] Validation started with lhcb-gaussino-dev#15
• [2023-02-02 00:05] Validation started with lhcb-gaussino#1433
• [2023-02-02 00:05] Automatic merge failed in [lhcb-g4-dev#1473](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1473/Geant4/checkout
• [2023-02-02 00:06] Automatic merge failed in [lhcb-gaussino-fastsim#472](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/472/Geant4/checkout
• [2023-02-02 00:06] Validation started with lhcb-gauss-lamarr#713
• [2023-02-02 00:07] Validation started with lhcb-gauss-dev#3420
• [2023-02-02 00:10] Validation started with lhcb-dd4hep#1391
• [2023-02-02 00:10] Validation started with lhcb-gaudi-head#3464
• [2023-02-02 00:14] Validation started with lhcb-head#3484
• [2023-02-02 00:15] Validation started with lhcb-lcg-dev3lhcb#49
• [2023-02-02 00:15] Validation started with lhcb-lcg-dev4lhcb#49
• [2023-02-02 00:18] Validation started with lhcb-sim10-gaudi-head#151
• [2023-02-02 00:21] Validation started with lhcb-head-2#772
• [2023-02-02 00:22] Validation started with lhcb-sim10#1448
• [2023-02-02 00:23] Validation started with lhcb-run3-cleanup#733
• [2023-02-03 00:05] Automatic merge failed in [lhcb-g4-dev#1474](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1474/Geant4/checkout
• [2023-02-03 00:05] Validation started with lhcb-gauss-lamarr#714
• [2023-02-03 00:05] Validation started with lhcb-gauss-gen-dev#1191
• [2023-02-03 00:05] Validation started with lhcb-gauss-dev#3421
• [2023-02-03 00:05] Automatic merge failed in [lhcb-gaussino-fastsim#473](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/473/Geant4/checkout
• [2023-02-03 00:05] Validation started with lhcb-gaussino#1434
• [2023-02-03 00:11] Validation started with lhcb-gaudi-head#3465
• [2023-02-03 00:11] Validation started with lhcb-dd4hep#1392
• [2023-02-03 00:12] Validation started with lhcb-head-2#773
• [2023-02-03 00:15] Validation started with lhcb-head#3485
• [2023-02-03 00:16] Validation started with lhcb-lcg-dev3lhcb#50
• [2023-02-03 00:21] Validation started with lhcb-sim10-gaudi-head#152
• [2023-02-03 00:22] Validation started with lhcb-sim10#1449
• [2023-02-03 00:30] Validation started with lhcb-run3-cleanup#734
• [2023-02-03 13:10] Validation started with lhcb-gaussino-dev#17
• [2023-02-03 13:46] Validation started with lhcb-gaussino-dev#18
• [2023-02-03 16:36] Validation started with lhcb-gaussino#1435
• [2023-02-03 16:47] Validation started with lhcb-gaussino#1436
• [2023-02-04 00:05] Automatic merge failed in [lhcb-g4-dev#1475](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1475/Geant4/checkout
• [2023-02-04 00:05] Validation started with lhcb-gauss-lamarr#715
• [2023-02-04 00:05] Validation started with lhcb-gauss-gen-dev#1192
• [2023-02-04 00:06] Validation started with lhcb-gauss-dev#3422
• [2023-02-04 00:07] Automatic merge failed in [lhcb-gaussino-fastsim#474](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/474/Geant4/checkout
• [2023-02-04 00:09] Validation started with lhcb-gaussino#1437
• [2023-02-04 00:11] Validation started with lhcb-gaudi-head#3466
• [2023-02-04 00:12] Validation started with lhcb-dd4hep#1393
• [2023-02-04 00:17] Validation started with lhcb-head-2#774
• [2023-02-04 00:19] Validation started with lhcb-head#3486
• [2023-02-04 00:20] Validation started with lhcb-lcg-dev3lhcb#51
• [2023-02-04 00:21] Validation started with lhcb-lcg-dev4lhcb#51
• [2023-02-04 00:29] Validation started with lhcb-sim10-gaudi-head#153
• [2023-02-04 00:31] Validation started with lhcb-sim10#1450
• [2023-02-04 00:33] Validation started with lhcb-run3-cleanup#735
• [2023-02-04 12:16] Validation started with lhcb-gaussino#1438
• [2023-02-04 12:58] Validation started with lhcb-gaussino#1439
• [2023-02-04 13:44] Validation started with lhcb-gaussino#1440
• [2023-02-04 20:56] Validation started with lhcb-gaussino#1441
• [2023-02-04 21:03] Validation started with lhcb-gaussino#1441
• [2023-02-04 21:10] Validation started with lhcb-gaussino#1441
• [2023-02-05 00:04] Automatic merge failed in [lhcb-g4-dev#1476](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1476/Geant4/checkout
• [2023-02-05 00:05] Validation started with lhcb-gauss-dev#3423
• [2023-02-05 00:05] Validation started with lhcb-gaussino#1442
• [2023-02-05 00:05] Automatic merge failed in [lhcb-gaussino-fastsim#475](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/475/Geant4/checkout
• [2023-02-05 00:09] Validation started with lhcb-gaudi-head#3467
• [2023-02-05 00:09] Validation started with lhcb-head-2#775
• [2023-02-05 00:09] Validation started with lhcb-dd4hep#1394
• [2023-02-05 00:10] Validation started with lhcb-head#3487
• [2023-02-05 00:13] Validation started with lhcb-lcg-dev3lhcb#52
• [2023-02-05 00:15] Validation started with lhcb-lcg-dev4lhcb#52
• [2023-02-05 00:17] Validation started with lhcb-sim10-gaudi-head#154
• [2023-02-05 00:18] Validation started with lhcb-sim10#1451
• [2023-02-05 00:26] Validation started with lhcb-run3-cleanup#736
• [2023-02-06 00:04] Automatic merge failed in [lhcb-g4-dev#1477](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1477/Geant4/checkout
• [2023-02-06 00:05] Validation started with lhcb-gauss-dev#3424
• [2023-02-06 00:05] Validation started with lhcb-gaussino#1443
• [2023-02-06 00:09] Validation started with lhcb-gaudi-head#3468
• [2023-02-06 00:09] Validation started with lhcb-head-2#776
• [2023-02-06 00:09] Validation started with lhcb-head#3488
• [2023-02-06 00:09] Validation started with lhcb-dd4hep#1395
• [2023-02-06 00:11] Validation started with lhcb-lcg-dev4lhcb#53
• [2023-02-06 00:14] Validation started with lhcb-run3-cleanup#737
• [2023-02-06 00:16] Validation started with lhcb-sim10-gaudi-head#155
• [2023-02-06 00:16] Validation started with lhcb-sim10#1452
• [2023-02-06 11:16] Validation started with lhcb-gaussino#1444
• [2023-02-06 12:12] Validation started with lhcb-gaussino#1445
• [2023-02-06 16:06] Validation started with lhcb-gaussino#1446
• [2023-02-07 00:04] Automatic merge failed in [lhcb-g4-dev#1478](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1478/Geant4/checkout
• [2023-02-07 00:05] Validation started with lhcb-gauss-gen-dev#1193
• [2023-02-07 00:05] Validation started with lhcb-gauss-dev#3425
• [2023-02-07 00:06] Validation started with lhcb-gauss-lamarr#716
• [2023-02-07 00:07] Automatic merge failed in [lhcb-gaussino-fastsim#477](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/477/Geant4/checkout
• [2023-02-07 00:08] Validation started with lhcb-gaussino#1447
• [2023-02-07 00:09] Validation started with lhcb-gaudi-head#3469
• [2023-02-07 00:10] Validation started with lhcb-dd4hep#1396
• [2023-02-07 00:16] Validation started with lhcb-head-2#777
• [2023-02-07 00:17] Validation started with lhcb-head#3489
• [2023-02-07 00:20] Validation started with lhcb-lcg-dev4lhcb#54
• [2023-02-07 00:25] Validation started with lhcb-lcg-dev3lhcb#54
• [2023-02-07 00:29] Validation started with lhcb-sim10-gaudi-head#156
• [2023-02-07 00:29] Validation started with lhcb-sim10#1453
• [2023-02-07 00:37] Validation started with lhcb-run3-cleanup#738
• [2023-02-07 11:28] Validation started with lhcb-gaussino#1448
• [2023-02-07 14:32] Validation started with lhcb-gaussino#1449
• [2023-02-07 17:34] Validation started with lhcb-gaussino#1450
• [2023-02-08 00:04] Automatic merge failed in [lhcb-g4-dev#1479](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1479/Geant4/checkout
• [2023-02-08 00:04] Validation started with lhcb-gaussino#1451
• [2023-02-08 00:05] Validation started with lhcb-gauss-dev#3426
• [2023-02-08 00:09] Validation started with lhcb-gaudi-head#3470
• [2023-02-08 00:09] Validation started with lhcb-head#3490
• [2023-02-08 00:09] Validation started with lhcb-lcg-dev3lhcb#55
• [2023-02-08 00:09] Validation started with lhcb-lcg-dev4lhcb#55
• [2023-02-08 00:09] Validation started with lhcb-dd4hep#1397
• [2023-02-08 00:09] Validation started with lhcb-head-2#778
• [2023-02-08 00:14] Validation started with lhcb-run3-cleanup#739
• [2023-02-08 00:15] Validation started with lhcb-sim10-gaudi-head#157
• [2023-02-08 00:15] Validation started with lhcb-sim10#1454
• [2023-02-09 00:04] Automatic merge failed in [lhcb-g4-dev#1480](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-g4-dev/1480/Geant4/checkout
• [2023-02-09 00:05] Validation started with lhcb-gauss-gen-dev#1194
• [2023-02-09 00:05] Validation started with lhcb-gauss-dev#3427
• [2023-02-09 00:06] Validation started with lhcb-gauss-lamarr#717
• [2023-02-09 00:07] Automatic merge failed in [lhcb-gaussino-fastsim#478](https://lhcb-nightlies.web.cern.ch/nightly/lhcb-gaussino-fastsim/478/Geant4/checkout
• [2023-02-09 00:08] Validation started with lhcb-gaussino#1452
• [2023-02-09 00:09] Validation started with lhcb-dd4hep#1398
• [2023-02-09 00:10] Validation started with lhcb-gaudi-head#3471
• [2023-02-09 00:15] Validation started with lhcb-head-2#779
• [2023-02-09 00:16] Validation started with lhcb-head#3491
• [2023-02-09 00:21] Validation started with lhcb-lcg-dev4lhcb#56
• [2023-02-09 00:21] Validation started with lhcb-lcg-dev3lhcb#56
• [2023-02-09 00:29] Validation started with lhcb-sim10-gaudi-head#158
• [2023-02-09 00:33] Validation started with lhcb-sim10#1455
• [2023-02-09 00:36] Validation started with lhcb-run3-cleanup#740

For the record, a brief update from #10 (closed). In my private tests, this patch now clearly a shows a speedup by a factor two (x2.0) for the Gauss simulation workflow in the sim10 run3 configuration used for HEP-SCORE.

My previous comment that there is no clear eveidence of a speedup was based on incorrect tests with a configuration bug (I was essentially comparing the old code to the old code - hence any differences were only due to fluctuations).

added all-slots label

removed 1 deleted label

added enhancement label

This is the only MR needed for a release of Geant4 10.6.2.7 (v10r6p2t7) for Sim10.

It has to be included in any release of the G4 10.6 to be used for Gauss-on-Gaussino with GEANT4_BUILD_MULTITHREADED=True

But it will be to be reverted for G4 10.7

cc: @dpopov, @clemenci, @mimazure, @admorris

changed milestone to %Geant4 v10r6p2t8

changed milestone to %v10r6p2t7

mentioned in issue #11 (closed)

marked this merge request as ready

assigned to @dpopov

added to be removed label

removed to be removed label

mentioned in commit 220d9c2f

merged