Follow-up from "Refactoring of the track class to have a flexible starting point to implement other fitting routines"
The following discussions from !189 (merged) should be addressed:
-
@simonspa started a discussion: (+3 comments) Dito, see above. Make sure they should be understandable without reading the source code.
-
@simonspa started a discussion: (+2 comments) we have
cluster->errorX()
andcluster->errorY()
- don't we have to use them here? -
@simonspa started a discussion: (+1 comment) Can't you simply use Eigen3 matrices?
-
@simonspa started a discussion: (+1 comment) That would make this here
detx.invert()
-
@simonspa started a discussion: (+2 comments) I don't like setting non-sensible defaults
-
@simonspa started a discussion: (+4 comments) what happens if this is not set?
-
@simonspa started a discussion: I will fix these later