Skip to content

Define parameter(s) to restrict some of the possible module pairs when setting the detection efficiencies - make the set-up of detection efficiencies optional #28

@nkarakatsanis

Description

@nkarakatsanis

I would suggest we add a restiction parameter definition (in JSON files as well).
For example to restrict coincidences in xy planes:
scannerGeometry.min_rsec_xy_diff=2 //minimum rsector index difference allowed in coincidence in xy (unsigned int)
Then when we calculate num_SGIDs, we could define:
assert (min_rsec_xy_diff>=0 && min_rsec_xy_diff<n_rsec_xy);
unsigned int n_rsec_xy_fan; //total number of modules allowed in coincidence with a given module in xy at a given angle
if scannerGeometry.min_rsec_xy_diff==1 n_rsec_xy_fan=scannerGeometry.n_rsec_xy-1
else n_rsec_xy_fan=scannerGeometry.n_rsec_xy-2*scannerGeometry.min_rsec_xy_diff;
const unsigned int num_SGIDs = scannerGeometry.n_rsec_z * scannerGeometry.n_rsec_z * n_rsec_xy_fan;

I find more intuitive to ask a user for the parameter "min_rsec_xy_diff" and then calculate "n_rsec_xy_fan" as opposed to directly asking for "n_rsec_xy_fan".

We also need to consider a method to also restrict the size of the SGID LUT accordingly as it is determined by the number of possible module pairs.

Furthermore, we may need to make optional the set of detection efficiencies given the relatively large size of the header file if not enough symmetries are considered. This could be implemeted as a cmd option.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions