Review of MagneticField class and field map reading in Cyclotron class
Split from issue #70 (closed).
The class MagneticField has the task to read a cyclotron field map for the MatchedGauss
distribution. It does this by inheriting and duplicating code from the Cyclotron
class.
This seems not ideal to me. So I am wondering if we can split off the field map reading from the Cyclotron class?
This would
- split off functionality into a separate class
- avoid duplication
- reduce the large Cyclotron class
- (one could use a factory-like pattern to produce the fieldmaps).