Project

General

Profile

Feature #25432

Incorporate and register a custom physics list

Added by David Rivera about 1 month ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Start date:
01/25/2021
Due date:
% Done:

100%

Estimated time:
Duration:

Description

I have added a custom physics list to artg4tk and placed it in a new directory *artg4tk/artg4tk/lists*

This custom physics list (under the name *MyQGSP_BERT_ArHP*) is registered as an option and can be selected within the PhysicsList service configuration.
The custom list was added to acommodate a request from the UC Davis working group in order to base the neutron capture gammas on the latest NNDC data for Ar40.
Again, this is optional and does not constitute a breaking change.

This is implemented in my feature branch : feature/drivera_custom_argon_ncapture 
It is required for the larg4 pull request found here : https://github.com/LArSoft/larg4/pull/21

History

#1 Updated by Lynn Garren about 1 month ago

  • Assignee changed from David Rivera to Lynn Garren
  • Status changed from Under Discussion to Assigned

David, I presume that you discussed this with Hans and the geant4 team is OK with the proposed change, right?

#2 Updated by Lynn Garren about 1 month ago

Also, this is not "done" until a release is tagged which incorporates this change.

#3 Updated by David Rivera about 1 month ago

  • % Done changed from 100 to 90

Lynn Garren wrote:

David, I presume that you discussed this with Hans and the geant4 team is OK with the proposed change, right?

I discussed it with Hans and Robert Hatcher quite some time ago (probably in 2019).
I have been using custom physics lists for my own purposes, and since I got a request from a working group to incorporate their custom code,
I went ahead and made this version of it for them.

For the time being, I thought it would be cleaner to put the custom list in artg4tk, instead of in larg4, since the physic list service is defined in artg4tk.
We can discuss it further if needed. The latter would introduce a dependency of artg4tk on larg4, which I wanted to avoid.
The other way around, where larg4 depends on artg4tk, is more natural and is already in place.

This is not really a "change", per-se -- it's more an addition.

#4 Updated by Lynn Garren about 1 month ago

David, thank you for the clarification. Also, it is a very good thing you did it this way, since a dependency of artg4tk on larg4 is not allowed. ;)

#5 Updated by Lynn Garren about 1 month ago

  • % Done changed from 90 to 100
  • Status changed from Assigned to Resolved

artg4tk v10_00_03 is now built and installed in the cvmfs larsoft distribution.

#6 Updated by Lynn Garren about 1 month ago

  • Status changed from Resolved to Closed

FYI, this is also in artg4tk v10_02_00 for the upcoming larsoft release with art 3.06.

Also available in: Atom PDF