GilesStrong / tomopt

TomOpt: Differential Muon Tomography Optimisation
GNU Affero General Public License v3.0
4 stars 0 forks source link

Automatic detector-panel removal #169

Open GilesStrong opened 1 year ago

GilesStrong commented 1 year ago

 ## Problem

The optimisation will sometimes try to minimiser some panels, in order to assign their budget to other, more useful panels, however, this can lead problems, e.g. ScatterBatch._filter_scatters will remove any muons with at least one high-uncertainty hit, i.e. all muons get removed. Example:

Screenshot 2022-12-12 at 11 56 48

Potential solution

A callback that automatically removes panels. This could trigger on e.g.

GilesStrong commented 1 year ago

Potentially this could also be caused by an anomalous gradient making one of the panels jump in assigned budget; may need a step-limiting callback.