Skip to content
Snippets Groups Projects

RadiationMapsMaker: added maps of mass fractions for all natural elements

Merged Sven Menke requested to merge menke/athena:21.0-RadMaps-add-massFractions into 21.0

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • resolved all threads

  • No outstanding issue at level 1, moving forward to level 2 for approval.

  • Pavol Strizenec
  • Pavol Strizenec
  • Author Developer

    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

  • Dear Sven,

    I'm not an expert at all here, as I said, such instructions are given to the shifters. Escalating to the experts level, @emoyse or @wlampl should say.....

      Best,
               Pavol [as L2 MR shifter]
  • Author Developer

    Dear all,

    what's the status of this MR? The code is needed for the upgrade simulation and it's 6 days old now ...

    Thanks & Cheers, --Sven

  • Copyrights are changed for substantial changes in the code and I guess shifters decided yours count as such. The change is not critical though but let me ping @emoyse and @wlampl again.

  • Sven Menke added 1 commit

    added 1 commit

    • a2bb884e - RadiationMapsMaker: update/add copyright lines

    Compare with previous version

  • This merge request affects 1 package:

    • Simulation/G4Utilities/G4UserActions

    Adding @jchapman ,@vpascuzz as watchers

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading