Problem 2418 - Bug due to changes in cylinder scoring mesh in 11.0 beta
Summary: Bug due to changes in cylinder scoring mesh in 11.0 beta
Status: RESOLVED FIXED
Alias: None
Product: Geant4
Classification: Unclassified
Component: digits_hits (show other problems)
Version: other
Hardware: All All
: P4 major
Assignee: asai
URL:
Depends on:
Blocks:
 
Reported: 2021-08-25 15:12 CEST by Dominik Werthmueller
Modified: 2021-09-24 18:35 CEST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this problem.
Description Dominik Werthmueller 2021-08-25 15:12:20 CEST 
Hi

There seems to be a bug related to the extension of the cylindrical mesh being introduced with Version 11.0 Beta. It has not been fixed yet on the public gitlab.

The ordering of the cylinder mesh size components in G4ScoringCylinder::fSize has been changed in 11.0 in order to store Rmin. Rmax, dZ. Those changes were not accounted for in G4ScoreQuantityMessenger, where the cylinder size is set:

317: pps->SetCylinderSize(msize[0], msize[1]);  // given in dr dz
352: pps->SetCylinderSize(msize[0], msize[1]);  // given in dr dz
405: pps->SetCylinderSize(msize[0], msize[1]);  // given in dr dz

These should be changed to

pps->SetCylinderSize(msize[1], msize[2]);

Probably also the G4PS*Cylinder3D classes need some refactoring because of Rmin?

Cheers
Dominik
Comment 1 asai 2021-09-24 18:35:58 CEST 
Thanks for reporting this.
It will be fixed in the coming 11.0 release.