RadiationMapsMaker: added maps of mass fractions for all natural elements
each grid point will collect the density- and step-length-weighted mass-fraction of the encountered elements. This gives the effective atomic composition of materials in each grid point. The first and last element to track is configurable - by default all natural elements ( 1 <= Z <= 92 ) are enabled. Example output in the service area of LAr from a geantino scan in the barrel: H: 0.00525763 C: 0.162227 N: 0.0470331 O: 0.0583856 F: 0.0808482 Si: 0.000496895 Ar: 0.000285226 Mn: 0.000496895 Fe: 0.0486957 Cu: 0.596274
Merge request reports
Activity
added 21.0 Simulation labels
added review-pending-level-1 label
CI Result SUCCESS (hash 5267b5dc)Athena AthDataQuality AthSimulation externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthDataQuality: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 41719]- Resolved by Edward Moyse
added review-pending-level-2 label and removed review-pending-level-1 label
- Resolved by Edward Moyse
- Resolved by Edward Moyse
added review-user-action-required label
added alsoTargeting:21.3 alsoTargeting:21.9 labels
Dear @pavol,
I've answered to the request and thus believe I fulfilled the obligations connected to the review-user-action-required flag.
Please mark the discussion as resolved - or find a person trained in legal issues to explain how copyright statements in C++ comment lines are supposed to be handled. I - as mentioned before - am not competent in copyright law and can not edit lines I do not know the syntax of and have no way of validating. If interpreted in plain English the statement is wrong in many ways. None of the parts have been written at CERN, the author of the code (me) never consented to transfer of copyright to third party (CERN), is not mentioned in the code and if the numbers 2002 and 2017 are supposed to reflect years - the range prior to 2017 is not correct since the code did not exist then.
The code in question is needed also in 21.3 and 21.9 - and I've just added the respective flags. Automatic sweeps should work here ... For master I will do a manual sweep since the API there is different.
Thanks & Cheers, --Sven
added review-pending-expert label and removed review-pending-level-2 label
added 1 commit
- a2bb884e - RadiationMapsMaker: update/add copyright lines
added review-pending-level-1 label and removed review-pending-expert review-user-action-required labels